622665 - Remove old unused na2 import code

Status: RESOLVED FIXED

(MailNews Core :: Import, defect)

Description

[Mark Banner (:standard8)]

Attachment: The fix

I found some old na2 import code the other day. We never hooked it up to import, so we should just remove it, this additionally simplifies some of the interfaces (because we no longer need to worry about if addresses are home or work based) and means we can drop some strings.

Comment 1

[[e-mail]]

Comment on attachment 500884 [details] [diff] [review]
The fix

>-    return mailInterface.BeginImport( success, error, false) ? true : false;
>+    return mailInterface.BeginImport(success, error) ? true : false;

>+    if (addInterface.BeginImport(success, error))
>+      return true;
>+    return false;

Nit: since BeginImport is already boolean, no need to use if or ?: on them.

Comment 2

[Mark Banner (:standard8)]

Checked in with comment addressed: http://hg.mozilla.org/comm-central/rev/12ea69f572c6