Re: PSA: mozilla/unused.h renamed to mozilla/Unused.h

2016-08-27 Thread Tim Guan-tin Chien
Hi :gps, The more I read into this, the more I think the hook might not likely fix things. The hook would reject rename-only commits, but the hook would likely not reject a rename commit from |foo| to |bar| and another rename commit from |bar| to |Foo|. These two commits, as you pointed out,

Re: PSA: mozilla/unused.h renamed to mozilla/Unused.h

2016-08-27 Thread Mike Conley
Yes, thank you Kan-Ru! On Aug 27, 2016 2:10 PM, "Nick Fitzgerald" wrote: > Thanks Kan-Ru! I've personally tried to include the header with the wrong > case multiple times, so I appreciate this change a lot. > > On Fri, Aug 26, 2016 at 7:54 PM, Kan-Ru Chen

Re: PSA: mozilla/unused.h renamed to mozilla/Unused.h

2016-08-27 Thread Nick Fitzgerald
Thanks Kan-Ru! I've personally tried to include the header with the wrong case multiple times, so I appreciate this change a lot. On Fri, Aug 26, 2016 at 7:54 PM, Kan-Ru Chen wrote: > Hello, > > In Bug 1297276 I landed a patch to rename mozilla/unused.h to > mozilla/Unused.h

Re: PSA: mozilla/unused.h renamed to mozilla/Unused.h

2016-08-27 Thread Gregory Szorc
On Sat, Aug 27, 2016 at 9:00 AM, Eric Rescorla wrote: > > On Sat, Aug 27, 2016 at 8:53 AM, Gregory Szorc wrote: > >> >> >> > On Aug 27, 2016, at 07:09, Kan-Ru Chen wrote: >> > >> >> On Sat, Aug 27, 2016, at 11:35 AM, Gregory Szorc wrote: >>

Re: PSA: mozilla/unused.h renamed to mozilla/Unused.h

2016-08-27 Thread Eric Rescorla
On Sat, Aug 27, 2016 at 8:53 AM, Gregory Szorc wrote: > > > > On Aug 27, 2016, at 07:09, Kan-Ru Chen wrote: > > > >> On Sat, Aug 27, 2016, at 11:35 AM, Gregory Szorc wrote: > >>> On Fri, Aug 26, 2016 at 8:27 PM, Steve Fink wrote: > >>>

Re: PSA: mozilla/unused.h renamed to mozilla/Unused.h

2016-08-27 Thread Gregory Szorc
> On Aug 27, 2016, at 07:09, Kan-Ru Chen wrote: > >> On Sat, Aug 27, 2016, at 11:35 AM, Gregory Szorc wrote: >>> On Fri, Aug 26, 2016 at 8:27 PM, Steve Fink wrote: >>> On 08/26/2016 08:16 PM, Gregory Szorc wrote: > On Aug 26, 2016,

Re: PSA: mozilla/unused.h renamed to mozilla/Unused.h

2016-08-27 Thread Andrew McCreight
On Fri, Aug 26, 2016 at 8:16 PM, Gregory Szorc wrote: > What I'm trying to say is thank you for reminding me to implement the > hook. And congratulations on likely being the last person to perform a case > only rename on the repo. > Sarcastic commentary on an innocent

Re: What is the big picture view of where the Mozilla/Firefox platform is going?

2016-08-27 Thread Dirkjan Ochtman
On Fri, Aug 26, 2016 at 9:12 PM, Bobby Holley wrote: > My 1000-ft take on big-picture direction: > * Platform wants to pivot from implementing Platform things in C++ (which > is prone to memory hazards) and JS (which is weakly typed, has runtime > overhead, and various

Re: PSA: mozilla/unused.h renamed to mozilla/Unused.h

2016-08-27 Thread Kan-Ru Chen
On Sat, Aug 27, 2016, at 11:35 AM, Gregory Szorc wrote: > On Fri, Aug 26, 2016 at 8:27 PM, Steve Fink wrote: > > > On 08/26/2016 08:16 PM, Gregory Szorc wrote: > > > >> > >> On Aug 26, 2016, at 19:54, Kan-Ru Chen wrote: > >>> > >>> Hello, > >>> > >>> In Bug