| Index: nss/lib/base/item.c
|
| diff --git a/nss/lib/base/item.c b/nss/lib/base/item.c
|
| index dd463dcf92c9c97b0274d56b2421fa44bc8c4ed9..a1bb802ed6a29133344364330dacd0076d8ebc9e 100644
|
| --- a/nss/lib/base/item.c
|
| +++ b/nss/lib/base/item.c
|
| @@ -22,78 +22,69 @@
|
| * NSS_ERROR_NO_MEMORY
|
| * NSS_ERROR_ARENA_MARKED_BY_ANOTHER_THREAD
|
| * NSS_ERROR_INVALID_POINTER
|
| - *
|
| + *
|
| * Return value:
|
| * A pointer to an NSSItem upon success
|
| * NULL upon failure
|
| */
|
|
|
| NSS_IMPLEMENT NSSItem *
|
| -nssItem_Create
|
| -(
|
| - NSSArena *arenaOpt,
|
| - NSSItem *rvOpt,
|
| - PRUint32 length,
|
| - const void *data
|
| -)
|
| +nssItem_Create(NSSArena *arenaOpt, NSSItem *rvOpt, PRUint32 length,
|
| + const void *data)
|
| {
|
| - NSSItem *rv = (NSSItem *)NULL;
|
| + NSSItem *rv = (NSSItem *)NULL;
|
|
|
| #ifdef DEBUG
|
| - if( (NSSArena *)NULL != arenaOpt ) {
|
| - if( PR_SUCCESS != nssArena_verifyPointer(arenaOpt) ) {
|
| - return (NSSItem *)NULL;
|
| + if ((NSSArena *)NULL != arenaOpt) {
|
| + if (PR_SUCCESS != nssArena_verifyPointer(arenaOpt)) {
|
| + return (NSSItem *)NULL;
|
| + }
|
| }
|
| - }
|
|
|
| - if( (const void *)NULL == data ) {
|
| - if( length > 0 ) {
|
| - nss_SetError(NSS_ERROR_INVALID_POINTER);
|
| - return (NSSItem *)NULL;
|
| + if ((const void *)NULL == data) {
|
| + if (length > 0) {
|
| + nss_SetError(NSS_ERROR_INVALID_POINTER);
|
| + return (NSSItem *)NULL;
|
| + }
|
| }
|
| - }
|
| #endif /* DEBUG */
|
|
|
| - if( (NSSItem *)NULL == rvOpt ) {
|
| - rv = (NSSItem *)nss_ZNEW(arenaOpt, NSSItem);
|
| - if( (NSSItem *)NULL == rv ) {
|
| - goto loser;
|
| + if ((NSSItem *)NULL == rvOpt) {
|
| + rv = (NSSItem *)nss_ZNEW(arenaOpt, NSSItem);
|
| + if ((NSSItem *)NULL == rv) {
|
| + goto loser;
|
| + }
|
| + } else {
|
| + rv = rvOpt;
|
| }
|
| - } else {
|
| - rv = rvOpt;
|
| - }
|
|
|
| - rv->size = length;
|
| - rv->data = nss_ZAlloc(arenaOpt, length);
|
| - if( (void *)NULL == rv->data ) {
|
| - goto loser;
|
| - }
|
| + rv->size = length;
|
| + rv->data = nss_ZAlloc(arenaOpt, length);
|
| + if ((void *)NULL == rv->data) {
|
| + goto loser;
|
| + }
|
|
|
| - if( length > 0 ) {
|
| - (void)nsslibc_memcpy(rv->data, data, length);
|
| - }
|
| + if (length > 0) {
|
| + (void)nsslibc_memcpy(rv->data, data, length);
|
| + }
|
|
|
| - return rv;
|
| + return rv;
|
|
|
| - loser:
|
| - if( rv != rvOpt ) {
|
| - nss_ZFreeIf(rv);
|
| - }
|
| +loser:
|
| + if (rv != rvOpt) {
|
| + nss_ZFreeIf(rv);
|
| + }
|
|
|
| - return (NSSItem *)NULL;
|
| + return (NSSItem *)NULL;
|
| }
|
|
|
| NSS_IMPLEMENT void
|
| -nssItem_Destroy
|
| -(
|
| - NSSItem *item
|
| -)
|
| +nssItem_Destroy(NSSItem *item)
|
| {
|
| - nss_ClearErrorStack();
|
| -
|
| - nss_ZFreeIf(item->data);
|
| - nss_ZFreeIf(item);
|
| + nss_ClearErrorStack();
|
|
|
| + nss_ZFreeIf(item->data);
|
| + nss_ZFreeIf(item);
|
| }
|
|
|
| /*
|
| @@ -106,34 +97,29 @@ nssItem_Destroy
|
| * NSS_ERROR_NO_MEMORY
|
| * NSS_ERROR_ARENA_MARKED_BY_ANOTHER_THREAD
|
| * NSS_ERROR_INVALID_ITEM
|
| - *
|
| + *
|
| * Return value:
|
| * A pointer to an NSSItem upon success
|
| * NULL upon failure
|
| */
|
|
|
| NSS_IMPLEMENT NSSItem *
|
| -nssItem_Duplicate
|
| -(
|
| - NSSItem *obj,
|
| - NSSArena *arenaOpt,
|
| - NSSItem *rvOpt
|
| -)
|
| +nssItem_Duplicate(NSSItem *obj, NSSArena *arenaOpt, NSSItem *rvOpt)
|
| {
|
| #ifdef DEBUG
|
| - if( (NSSArena *)NULL != arenaOpt ) {
|
| - if( PR_SUCCESS != nssArena_verifyPointer(arenaOpt) ) {
|
| - return (NSSItem *)NULL;
|
| + if ((NSSArena *)NULL != arenaOpt) {
|
| + if (PR_SUCCESS != nssArena_verifyPointer(arenaOpt)) {
|
| + return (NSSItem *)NULL;
|
| + }
|
| }
|
| - }
|
|
|
| - if( (NSSItem *)NULL == obj ) {
|
| - nss_SetError(NSS_ERROR_INVALID_ITEM);
|
| - return (NSSItem *)NULL;
|
| - }
|
| + if ((NSSItem *)NULL == obj) {
|
| + nss_SetError(NSS_ERROR_INVALID_ITEM);
|
| + return (NSSItem *)NULL;
|
| + }
|
| #endif /* DEBUG */
|
|
|
| - return nssItem_Create(arenaOpt, rvOpt, obj->size, obj->data);
|
| + return nssItem_Create(arenaOpt, rvOpt, obj->size, obj->data);
|
| }
|
|
|
| #ifdef DEBUG
|
| @@ -151,18 +137,15 @@ nssItem_Duplicate
|
| */
|
|
|
| NSS_IMPLEMENT PRStatus
|
| -nssItem_verifyPointer
|
| -(
|
| - const NSSItem *item
|
| -)
|
| +nssItem_verifyPointer(const NSSItem *item)
|
| {
|
| - if( ((const NSSItem *)NULL == item) ||
|
| - (((void *)NULL == item->data) && (item->size > 0)) ) {
|
| - nss_SetError(NSS_ERROR_INVALID_ITEM);
|
| - return PR_FAILURE;
|
| - }
|
| + if (((const NSSItem *)NULL == item) ||
|
| + (((void *)NULL == item->data) && (item->size > 0))) {
|
| + nss_SetError(NSS_ERROR_INVALID_ITEM);
|
| + return PR_FAILURE;
|
| + }
|
|
|
| - return PR_SUCCESS;
|
| + return PR_SUCCESS;
|
| }
|
| #endif /* DEBUG */
|
|
|
| @@ -181,28 +164,23 @@ nssItem_verifyPointer
|
| */
|
|
|
| NSS_IMPLEMENT PRBool
|
| -nssItem_Equal
|
| -(
|
| - const NSSItem *one,
|
| - const NSSItem *two,
|
| - PRStatus *statusOpt
|
| -)
|
| +nssItem_Equal(const NSSItem *one, const NSSItem *two, PRStatus *statusOpt)
|
| {
|
| - if( (PRStatus *)NULL != statusOpt ) {
|
| - *statusOpt = PR_SUCCESS;
|
| - }
|
| + if ((PRStatus *)NULL != statusOpt) {
|
| + *statusOpt = PR_SUCCESS;
|
| + }
|
|
|
| - if( ((const NSSItem *)NULL == one) && ((const NSSItem *)NULL == two) ) {
|
| - return PR_TRUE;
|
| - }
|
| + if (((const NSSItem *)NULL == one) && ((const NSSItem *)NULL == two)) {
|
| + return PR_TRUE;
|
| + }
|
|
|
| - if( ((const NSSItem *)NULL == one) || ((const NSSItem *)NULL == two) ) {
|
| - return PR_FALSE;
|
| - }
|
| + if (((const NSSItem *)NULL == one) || ((const NSSItem *)NULL == two)) {
|
| + return PR_FALSE;
|
| + }
|
|
|
| - if( one->size != two->size ) {
|
| - return PR_FALSE;
|
| - }
|
| + if (one->size != two->size) {
|
| + return PR_FALSE;
|
| + }
|
|
|
| - return nsslibc_memequal(one->data, two->data, one->size, statusOpt);
|
| + return nsslibc_memequal(one->data, two->data, one->size, statusOpt);
|
| }
|
|
|