337942 - Remove some unused interface functions in address book

Status: RESOLVED FIXED

(MailNews Core :: Address Book, defect)

Description

[Mark Banner (:standard8)]

I've found the following interface functions that aren't used internally, and I can't see any real use externally either:

nsIAbDirectory::addMailListWithKey
nsIAddrDatabase::createMailListAndAddToDBWithKey

These aren't actually used anywhere, and from what I can tell never have been. Additionally I think its wrong as it exposes a back-end database thing (the key) to the front-end, and in this case is mork specific.

nsIAbDirectory::getValueForCard(in nsIAbCard card, in string name)
nsIAbDirectory::setValueForCard(in nsIAbCard card, in string name, in wstring value)

We have the nsIAbCard interface that will do this for us and the code that is there is mdb specific.

Comment 1

[Mark Banner (:standard8)]

Attachment: Remove unused functions

Simple patch to remove the unused functions as detailed in comment 0.

Comment 2

[Mark Banner (:standard8)]

Note to self: nsIAddrDatabase::createNewCardAndAddToDBWithKey can also be removed.

Comment 3

[Mark Banner (:standard8)]

Comment on attachment 221984 [details] [diff] [review]
Remove unused functions

I'm going to add the other function removal that I found to this patch before it gets reviewed...

Comment 4

[Mark Banner (:standard8)]

Attachment: Remove unused functions v2

Revised patch adding in the removal of the other redundant function I found.

Comment 5

[Mark Banner (:standard8)]

Comment on attachment 222051 [details] [diff] [review]
Remove unused functions v2

I'm guessing Dan is busy with calendar/lightning, so I'm switching reviews to David now he's back ;-)

Comment 6

[David :Bienvenu]

Comment on attachment 222051 [details] [diff] [review]
Remove unused functions v2

do we know that no extensions used get/set value for card? I guess they can use the card directly...

Comment 7

[Mark Banner (:standard8)]

(In reply to comment #6)
> (From update of attachment 222051 [details] [diff] [review] [edit])
> do we know that no extensions used get/set value for card? I guess they can use
> the card directly...
> 

I'm only putting this in for trunk onwards (not 1.8 branch) so extensions should have plenty of time to catch up. As they have the card to pass to the function, they should be able to just use the card straight off, I don't see any problems with that.

Comment 8

[Mark Banner (:standard8)]

Patch checked in. That's all for now on this bug -> fixed.