- From: Ehsan Akhgari <ehsan@mozilla.com>
- Date: Fri, 12 Aug 2011 18:11:54 -0400
On 11-08-12 6:10 PM, Ryosuke Niwa wrote: > On Fri, Aug 12, 2011 at 3:07 PM, Ehsan Akhgari<ehsan at mozilla.com> wrote: > >> On 11-08-09 6:36 PM, Jonas Sicking wrote: >>> >>> Sure, your API is more convenient in certain situations. But it also >>> encourages code duplication (I'll note that in the examples you >>> originally provided in this thread you always ended up duplicating >>> code between apply/reapply), which easily leads to bugs. >>> >> >> I think this is a very important point, and this downside makes me think >> that we shouldn't expose a reapply API. Since almost every case that I can >> think of where having a reapply function would make more sense requires >> authors to maintain other state information in their transaction objects, I >> don't think that maintaining one more boolean flag is going to make things >> noticeably harder for them. >> > > But having authors add flag in almost all cases isn't that nice either. Why do you think that authors need to specify the flag in almost all cases? Ehsan
Received on Friday, 12 August 2011 15:11:54 UTC