From 5cf13b9136fa6fbee8652479d0eb868b42a0e96f Mon Sep 17 00:00:00 2001 From: "chuang%netscape.com" Date: Fri, 4 Feb 2000 22:47:27 +0000 Subject: [PATCH] Bug 24877 Can't remove Additional email Address in Abook card, r=hangas --- mailnews/addrbook/src/nsAddrDatabase.cpp | 72 ++++++++++++------------ 1 file changed, 36 insertions(+), 36 deletions(-) diff --git a/mailnews/addrbook/src/nsAddrDatabase.cpp b/mailnews/addrbook/src/nsAddrDatabase.cpp index 3feddc61ffec..dc16d0e3815e 100644 --- a/mailnews/addrbook/src/nsAddrDatabase.cpp +++ b/mailnews/addrbook/src/nsAddrDatabase.cpp @@ -955,7 +955,7 @@ nsresult nsAddrDatabase::AddAttributeColumnsToRow(nsIAbCard *card, nsIMdbRow *ca char* pUTF8Str = nsnull; card->GetFirstName(&pUnicodeStr); unicharLength = nsCRT::strlen(pUnicodeStr); - if (pUnicodeStr && unicharLength) + if (pUnicodeStr) { INTL_ConvertFromUnicode(pUnicodeStr, unicharLength, (char**)&pUTF8Str); if (pUTF8Str) @@ -967,7 +967,7 @@ nsresult nsAddrDatabase::AddAttributeColumnsToRow(nsIAbCard *card, nsIMdbRow *ca PR_FREEIF(pUnicodeStr); card->GetLastName(&pUnicodeStr); unicharLength = nsCRT::strlen(pUnicodeStr); - if (pUnicodeStr && unicharLength) + if (pUnicodeStr) { INTL_ConvertFromUnicode(pUnicodeStr, unicharLength, (char**)&pUTF8Str); if (pUTF8Str) @@ -979,7 +979,7 @@ nsresult nsAddrDatabase::AddAttributeColumnsToRow(nsIAbCard *card, nsIMdbRow *ca PR_FREEIF(pUnicodeStr); card->GetDisplayName(&pUnicodeStr); unicharLength = nsCRT::strlen(pUnicodeStr); - if (pUnicodeStr && unicharLength) + if (pUnicodeStr) { INTL_ConvertFromUnicode(pUnicodeStr, unicharLength, (char**)&pUTF8Str); if (pUTF8Str) @@ -991,7 +991,7 @@ nsresult nsAddrDatabase::AddAttributeColumnsToRow(nsIAbCard *card, nsIMdbRow *ca PR_FREEIF(pUnicodeStr); card->GetNickName(&pUnicodeStr); unicharLength = nsCRT::strlen(pUnicodeStr); - if (pUnicodeStr && unicharLength) + if (pUnicodeStr) { INTL_ConvertFromUnicode(pUnicodeStr, unicharLength, (char**)&pUTF8Str); if (pUTF8Str) @@ -1003,7 +1003,7 @@ nsresult nsAddrDatabase::AddAttributeColumnsToRow(nsIAbCard *card, nsIMdbRow *ca PR_FREEIF(pUnicodeStr); card->GetPrimaryEmail(&pUnicodeStr); unicharLength = nsCRT::strlen(pUnicodeStr); - if (pUnicodeStr && unicharLength) + if (pUnicodeStr) { INTL_ConvertFromUnicode(pUnicodeStr, unicharLength, (char**)&pUTF8Str); if (pUTF8Str) @@ -1015,7 +1015,7 @@ nsresult nsAddrDatabase::AddAttributeColumnsToRow(nsIAbCard *card, nsIMdbRow *ca PR_FREEIF(pUnicodeStr); card->GetSecondEmail(&pUnicodeStr); unicharLength = nsCRT::strlen(pUnicodeStr); - if (pUnicodeStr && unicharLength) + if (pUnicodeStr) { INTL_ConvertFromUnicode(pUnicodeStr, unicharLength, (char**)&pUTF8Str); if (pUTF8Str) @@ -1032,7 +1032,7 @@ nsresult nsAddrDatabase::AddAttributeColumnsToRow(nsIAbCard *card, nsIMdbRow *ca card->GetWorkPhone(&pUnicodeStr); unicharLength = nsCRT::strlen(pUnicodeStr); - if (pUnicodeStr && unicharLength) + if (pUnicodeStr) { INTL_ConvertFromUnicode(pUnicodeStr, unicharLength, (char**)&pUTF8Str); if (pUTF8Str) @@ -1044,7 +1044,7 @@ nsresult nsAddrDatabase::AddAttributeColumnsToRow(nsIAbCard *card, nsIMdbRow *ca PR_FREEIF(pUnicodeStr); card->GetHomePhone(&pUnicodeStr); unicharLength = nsCRT::strlen(pUnicodeStr); - if (pUnicodeStr && unicharLength) + if (pUnicodeStr) { INTL_ConvertFromUnicode(pUnicodeStr, unicharLength, (char**)&pUTF8Str); if (pUTF8Str) @@ -1056,7 +1056,7 @@ nsresult nsAddrDatabase::AddAttributeColumnsToRow(nsIAbCard *card, nsIMdbRow *ca PR_FREEIF(pUnicodeStr); card->GetFaxNumber(&pUnicodeStr); unicharLength = nsCRT::strlen(pUnicodeStr); - if (pUnicodeStr && unicharLength) + if (pUnicodeStr) { INTL_ConvertFromUnicode(pUnicodeStr, unicharLength, (char**)&pUTF8Str); if (pUTF8Str) @@ -1068,7 +1068,7 @@ nsresult nsAddrDatabase::AddAttributeColumnsToRow(nsIAbCard *card, nsIMdbRow *ca PR_FREEIF(pUnicodeStr); card->GetPagerNumber(&pUnicodeStr); unicharLength = nsCRT::strlen(pUnicodeStr); - if (pUnicodeStr && unicharLength) + if (pUnicodeStr) { INTL_ConvertFromUnicode(pUnicodeStr, unicharLength, (char**)&pUTF8Str); if (pUTF8Str) @@ -1080,7 +1080,7 @@ nsresult nsAddrDatabase::AddAttributeColumnsToRow(nsIAbCard *card, nsIMdbRow *ca PR_FREEIF(pUnicodeStr); card->GetCellularNumber(&pUnicodeStr); unicharLength = nsCRT::strlen(pUnicodeStr); - if (pUnicodeStr && unicharLength) + if (pUnicodeStr) { INTL_ConvertFromUnicode(pUnicodeStr, unicharLength, (char**)&pUTF8Str); if (pUTF8Str) @@ -1092,7 +1092,7 @@ nsresult nsAddrDatabase::AddAttributeColumnsToRow(nsIAbCard *card, nsIMdbRow *ca PR_FREEIF(pUnicodeStr); card->GetHomeAddress(&pUnicodeStr); unicharLength = nsCRT::strlen(pUnicodeStr); - if (pUnicodeStr && unicharLength) + if (pUnicodeStr) { INTL_ConvertFromUnicode(pUnicodeStr, unicharLength, (char**)&pUTF8Str); if (pUTF8Str) @@ -1104,7 +1104,7 @@ nsresult nsAddrDatabase::AddAttributeColumnsToRow(nsIAbCard *card, nsIMdbRow *ca PR_FREEIF(pUnicodeStr); card->GetHomeAddress2(&pUnicodeStr); unicharLength = nsCRT::strlen(pUnicodeStr); - if (pUnicodeStr && unicharLength) + if (pUnicodeStr) { INTL_ConvertFromUnicode(pUnicodeStr, unicharLength, (char**)&pUTF8Str); if (pUTF8Str) @@ -1116,7 +1116,7 @@ nsresult nsAddrDatabase::AddAttributeColumnsToRow(nsIAbCard *card, nsIMdbRow *ca PR_FREEIF(pUnicodeStr); card->GetHomeCity(&pUnicodeStr); unicharLength = nsCRT::strlen(pUnicodeStr); - if (pUnicodeStr && unicharLength) + if (pUnicodeStr) { INTL_ConvertFromUnicode(pUnicodeStr, unicharLength, (char**)&pUTF8Str); if (pUTF8Str) @@ -1128,7 +1128,7 @@ nsresult nsAddrDatabase::AddAttributeColumnsToRow(nsIAbCard *card, nsIMdbRow *ca PR_FREEIF(pUnicodeStr); card->GetHomeState(&pUnicodeStr); unicharLength = nsCRT::strlen(pUnicodeStr); - if (pUnicodeStr && unicharLength) + if (pUnicodeStr) { INTL_ConvertFromUnicode(pUnicodeStr, unicharLength, (char**)&pUTF8Str); if (pUTF8Str) @@ -1140,7 +1140,7 @@ nsresult nsAddrDatabase::AddAttributeColumnsToRow(nsIAbCard *card, nsIMdbRow *ca PR_FREEIF(pUnicodeStr); card->GetHomeZipCode(&pUnicodeStr); unicharLength = nsCRT::strlen(pUnicodeStr); - if (pUnicodeStr && unicharLength) + if (pUnicodeStr) { INTL_ConvertFromUnicode(pUnicodeStr, unicharLength, (char**)&pUTF8Str); if (pUTF8Str) @@ -1152,7 +1152,7 @@ nsresult nsAddrDatabase::AddAttributeColumnsToRow(nsIAbCard *card, nsIMdbRow *ca PR_FREEIF(pUnicodeStr); card->GetHomeCountry(&pUnicodeStr); unicharLength = nsCRT::strlen(pUnicodeStr); - if (pUnicodeStr && unicharLength) + if (pUnicodeStr) { INTL_ConvertFromUnicode(pUnicodeStr, unicharLength, (char**)&pUTF8Str); if (pUTF8Str) @@ -1164,7 +1164,7 @@ nsresult nsAddrDatabase::AddAttributeColumnsToRow(nsIAbCard *card, nsIMdbRow *ca PR_FREEIF(pUnicodeStr); card->GetWorkAddress(&pUnicodeStr); unicharLength = nsCRT::strlen(pUnicodeStr); - if (pUnicodeStr && unicharLength) + if (pUnicodeStr) { INTL_ConvertFromUnicode(pUnicodeStr, unicharLength, (char**)&pUTF8Str); if (pUTF8Str) @@ -1176,7 +1176,7 @@ nsresult nsAddrDatabase::AddAttributeColumnsToRow(nsIAbCard *card, nsIMdbRow *ca PR_FREEIF(pUnicodeStr); card->GetWorkAddress2(&pUnicodeStr); unicharLength = nsCRT::strlen(pUnicodeStr); - if (pUnicodeStr && unicharLength) + if (pUnicodeStr) { INTL_ConvertFromUnicode(pUnicodeStr, unicharLength, (char**)&pUTF8Str); if (pUTF8Str) @@ -1188,7 +1188,7 @@ nsresult nsAddrDatabase::AddAttributeColumnsToRow(nsIAbCard *card, nsIMdbRow *ca PR_FREEIF(pUnicodeStr); card->GetWorkCity(&pUnicodeStr); unicharLength = nsCRT::strlen(pUnicodeStr); - if (pUnicodeStr && unicharLength) + if (pUnicodeStr) { INTL_ConvertFromUnicode(pUnicodeStr, unicharLength, (char**)&pUTF8Str); if (pUTF8Str) @@ -1200,7 +1200,7 @@ nsresult nsAddrDatabase::AddAttributeColumnsToRow(nsIAbCard *card, nsIMdbRow *ca PR_FREEIF(pUnicodeStr); card->GetWorkState(&pUnicodeStr); unicharLength = nsCRT::strlen(pUnicodeStr); - if (pUnicodeStr && unicharLength) + if (pUnicodeStr) { INTL_ConvertFromUnicode(pUnicodeStr, unicharLength, (char**)&pUTF8Str); if (pUTF8Str) @@ -1212,7 +1212,7 @@ nsresult nsAddrDatabase::AddAttributeColumnsToRow(nsIAbCard *card, nsIMdbRow *ca PR_FREEIF(pUnicodeStr); card->GetWorkZipCode(&pUnicodeStr); unicharLength = nsCRT::strlen(pUnicodeStr); - if (pUnicodeStr && unicharLength) + if (pUnicodeStr) { INTL_ConvertFromUnicode(pUnicodeStr, unicharLength, (char**)&pUTF8Str); if (pUTF8Str) @@ -1224,7 +1224,7 @@ nsresult nsAddrDatabase::AddAttributeColumnsToRow(nsIAbCard *card, nsIMdbRow *ca PR_FREEIF(pUnicodeStr); card->GetWorkCountry(&pUnicodeStr); unicharLength = nsCRT::strlen(pUnicodeStr); - if (pUnicodeStr && unicharLength) + if (pUnicodeStr) { INTL_ConvertFromUnicode(pUnicodeStr, unicharLength, (char**)&pUTF8Str); if (pUTF8Str) @@ -1236,7 +1236,7 @@ nsresult nsAddrDatabase::AddAttributeColumnsToRow(nsIAbCard *card, nsIMdbRow *ca PR_FREEIF(pUnicodeStr); card->GetJobTitle(&pUnicodeStr); unicharLength = nsCRT::strlen(pUnicodeStr); - if (pUnicodeStr && unicharLength) + if (pUnicodeStr) { INTL_ConvertFromUnicode(pUnicodeStr, unicharLength, (char**)&pUTF8Str); if (pUTF8Str) @@ -1248,7 +1248,7 @@ nsresult nsAddrDatabase::AddAttributeColumnsToRow(nsIAbCard *card, nsIMdbRow *ca PR_FREEIF(pUnicodeStr); card->GetDepartment(&pUnicodeStr); unicharLength = nsCRT::strlen(pUnicodeStr); - if (pUnicodeStr && unicharLength) + if (pUnicodeStr) { INTL_ConvertFromUnicode(pUnicodeStr, unicharLength, (char**)&pUTF8Str); if (pUTF8Str) @@ -1260,7 +1260,7 @@ nsresult nsAddrDatabase::AddAttributeColumnsToRow(nsIAbCard *card, nsIMdbRow *ca PR_FREEIF(pUnicodeStr); card->GetCompany(&pUnicodeStr); unicharLength = nsCRT::strlen(pUnicodeStr); - if (pUnicodeStr && unicharLength) + if (pUnicodeStr) { INTL_ConvertFromUnicode(pUnicodeStr, unicharLength, (char**)&pUTF8Str); if (pUTF8Str) @@ -1272,7 +1272,7 @@ nsresult nsAddrDatabase::AddAttributeColumnsToRow(nsIAbCard *card, nsIMdbRow *ca PR_FREEIF(pUnicodeStr); card->GetWebPage1(&pUnicodeStr); unicharLength = nsCRT::strlen(pUnicodeStr); - if (pUnicodeStr && unicharLength) + if (pUnicodeStr) { INTL_ConvertFromUnicode(pUnicodeStr, unicharLength, (char**)&pUTF8Str); if (pUTF8Str) @@ -1284,7 +1284,7 @@ nsresult nsAddrDatabase::AddAttributeColumnsToRow(nsIAbCard *card, nsIMdbRow *ca PR_FREEIF(pUnicodeStr); card->GetWebPage2(&pUnicodeStr); unicharLength = nsCRT::strlen(pUnicodeStr); - if (pUnicodeStr && unicharLength) + if (pUnicodeStr) { INTL_ConvertFromUnicode(pUnicodeStr, unicharLength, (char**)&pUTF8Str); if (pUTF8Str) @@ -1296,7 +1296,7 @@ nsresult nsAddrDatabase::AddAttributeColumnsToRow(nsIAbCard *card, nsIMdbRow *ca PR_FREEIF(pUnicodeStr); card->GetBirthYear(&pUnicodeStr); unicharLength = nsCRT::strlen(pUnicodeStr); - if (pUnicodeStr && unicharLength) + if (pUnicodeStr) { INTL_ConvertFromUnicode(pUnicodeStr, unicharLength, (char**)&pUTF8Str); if (pUTF8Str) @@ -1308,7 +1308,7 @@ nsresult nsAddrDatabase::AddAttributeColumnsToRow(nsIAbCard *card, nsIMdbRow *ca PR_FREEIF(pUnicodeStr); card->GetBirthMonth(&pUnicodeStr); unicharLength = nsCRT::strlen(pUnicodeStr); - if (pUnicodeStr && unicharLength) + if (pUnicodeStr) { INTL_ConvertFromUnicode(pUnicodeStr, unicharLength, (char**)&pUTF8Str); if (pUTF8Str) @@ -1320,7 +1320,7 @@ nsresult nsAddrDatabase::AddAttributeColumnsToRow(nsIAbCard *card, nsIMdbRow *ca PR_FREEIF(pUnicodeStr); card->GetBirthDay(&pUnicodeStr); unicharLength = nsCRT::strlen(pUnicodeStr); - if (pUnicodeStr && unicharLength) + if (pUnicodeStr) { INTL_ConvertFromUnicode(pUnicodeStr, unicharLength, (char**)&pUTF8Str); if (pUTF8Str) @@ -1332,7 +1332,7 @@ nsresult nsAddrDatabase::AddAttributeColumnsToRow(nsIAbCard *card, nsIMdbRow *ca PR_FREEIF(pUnicodeStr); card->GetCustom1(&pUnicodeStr); unicharLength = nsCRT::strlen(pUnicodeStr); - if (pUnicodeStr && unicharLength) + if (pUnicodeStr) { INTL_ConvertFromUnicode(pUnicodeStr, unicharLength, (char**)&pUTF8Str); if (pUTF8Str) @@ -1344,7 +1344,7 @@ nsresult nsAddrDatabase::AddAttributeColumnsToRow(nsIAbCard *card, nsIMdbRow *ca PR_FREEIF(pUnicodeStr); card->GetCustom2(&pUnicodeStr); unicharLength = nsCRT::strlen(pUnicodeStr); - if (pUnicodeStr && unicharLength) + if (pUnicodeStr) { INTL_ConvertFromUnicode(pUnicodeStr, unicharLength, (char**)&pUTF8Str); if (pUTF8Str) @@ -1356,7 +1356,7 @@ nsresult nsAddrDatabase::AddAttributeColumnsToRow(nsIAbCard *card, nsIMdbRow *ca PR_FREEIF(pUnicodeStr); card->GetCustom3(&pUnicodeStr); unicharLength = nsCRT::strlen(pUnicodeStr); - if (pUnicodeStr && unicharLength) + if (pUnicodeStr) { INTL_ConvertFromUnicode(pUnicodeStr, unicharLength, (char**)&pUTF8Str); if (pUTF8Str) @@ -1368,7 +1368,7 @@ nsresult nsAddrDatabase::AddAttributeColumnsToRow(nsIAbCard *card, nsIMdbRow *ca PR_FREEIF(pUnicodeStr); card->GetCustom4(&pUnicodeStr); unicharLength = nsCRT::strlen(pUnicodeStr); - if (pUnicodeStr && unicharLength) + if (pUnicodeStr) { INTL_ConvertFromUnicode(pUnicodeStr, unicharLength, (char**)&pUTF8Str); if (pUTF8Str) @@ -1380,7 +1380,7 @@ nsresult nsAddrDatabase::AddAttributeColumnsToRow(nsIAbCard *card, nsIMdbRow *ca PR_FREEIF(pUnicodeStr); card->GetNotes(&pUnicodeStr); unicharLength = nsCRT::strlen(pUnicodeStr); - if (pUnicodeStr && unicharLength) + if (pUnicodeStr) { INTL_ConvertFromUnicode(pUnicodeStr, unicharLength, (char**)&pUTF8Str); if (pUTF8Str)