Re: Intent to Unship: stream decoder for BinHex format

2017-11-14 Thread Shih-Chiang Chien
Lastest update: nsBinHexDecoder is going to be removed from both mozilla-central and comm-central. Patch for mozilla-central has been landed on Firefox 59 (see bug 1390708) and the work for comm-central will be tracked by Bug 1417187. Best Regards, Shih-Chiang Chien Mozilla Taiwan On Thu, Oct 26,

Re: Intent to Unship: stream decoder for BinHex format

2017-10-25 Thread Eric Shepherd (Sheppy)
Man… It’s weird that this makes me all nostalgic. :) Eric Shepherd Senior Technical Writer, MDN MDN: https://developer.mozilla.org/ Blog: https://www.bitstampede.com/ On October 18, 2017 at 8:50:20 AM, Joshua Cranmer 🐧 (pidgeo...@gmail.com) wrote: FWIW, I've considered ripping out the binhex de

Re: Intent to Unship: stream decoder for BinHex format

2017-10-18 Thread Joshua Cranmer 🐧
On 10/17/2017 10:45 AM, Boris Zbarsky wrote: On 10/17/17 5:47 AM, Shih-Chiang Chien wrote: I intent to remove all the code that handling BinHex decoding, i.e. nsBinHex.cpp, from mozilla-central if no other project is using it. The code was originally added for mailnews.  See https://bugzilla.

Re: Intent to Unship: stream decoder for BinHex format

2017-10-17 Thread Boris Zbarsky
On 10/17/17 8:12 PM, Myk Melez wrote: Shouldn't the code rather move to comm-central if it's specific to mailnews? That would be fine too, yes. -Boris ___ dev-platform mailing list dev-platform@lists.mozilla.org https://lists.mozilla.org/listinfo/dev

Re: Intent to Unship: stream decoder for BinHex format

2017-10-17 Thread Myk Melez
Boris Zbarsky 2017 October 17 at 07:45 Please double-check that this use case does not get broken, or condition the code to be mailnews-only or something. Shouldn't the code rather move to comm-central if it's specific to mailnews? -myk ___

Re: Intent to Unship: stream decoder for BinHex format

2017-10-17 Thread Boris Zbarsky
On 10/17/17 5:47 AM, Shih-Chiang Chien wrote: I intent to remove all the code that handling BinHex decoding, i.e. nsBinHex.cpp, from mozilla-central if no other project is using it. The code was originally added for mailnews. See https://bugzilla.mozilla.org/show_bug.cgi?id=81352 Please dou