Removal of nsILocalFile and other interfaces, was: Upcoming change to fixIterator function in iteratorUtils.jsm

Ben Bucksch ben.bucksch at beonex.com
Tue Aug 8 18:09:48 UTC 2017


I would suggest to keep such interfaces. an empty interface is not really a  maintenance burden. and adding this file will be 100 times easier than fixing all callers, even if it's just a replacement.

this is particularly true for extensions without maintainer. it's the user that counts, not who is at fault. this is a relatively easy way to avoid unnecessary breakage.

you might be able to just undo these m-c commits, or (to avoid changes to m-c) move the IDL files to a new dir in mailnews/compat/src/public/.

Ben

Am 7. August 2017 15:55:37 MESZ, schrieb "Joshua Cranmer 🐧" <pidgeot18 at gmail.com>:
>On 8/7/2017 8:53 AM, CardBook wrote:
>>
>> thanks a lot Jörg
>>
>> have you an example how to replace the nsILocalFile with nsIFile ?...
>>
>
>s/nsILocalFile/nsIFile/g. The contents of nsILocalFile were blanked out
>
>years ago and everything moved to nsIFile, but the interface itself was
>
>not deleted due solely to add-on impact.
>
>-- 
>Joshua Cranmer
>Thunderbird and DXR developer
>Source code archæologist
>
>_______________________________________________
>tb-planning mailing list
>tb-planning at mozilla.org
>https://mail.mozilla.org/listinfo/tb-planning
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.mozilla.org/pipermail/tb-planning/attachments/20170808/38283adb/attachment.html>


More information about the tb-planning mailing list