Please consider a CL just adding these curlies to already be LGTMed, so commit when ready without need for further feedback.
On Mon, Apr 15, 2013 at 2:35 PM, <[email protected]> wrote: > > https://codereview.appspot.**com/8612048/diff/16001/src/** > com/google/caja/ses/WeakMap.js<https://codereview.appspot.com/8612048/diff/16001/src/com/google/caja/ses/WeakMap.js> > File src/com/google/caja/ses/**WeakMap.js (right): > > https://codereview.appspot.**com/8612048/diff/16001/src/** > com/google/caja/ses/WeakMap.**js#newcode462<https://codereview.appspot.com/8612048/diff/16001/src/com/google/caja/ses/WeakMap.js#newcode462> > src/com/google/caja/ses/**WeakMap.js:462: if (typeof HostWeakMap === > 'function') (function() { > On 2013/04/15 21:33:30, MarkM wrote: > >> On 2013/04/15 21:19:55, kpreid2 wrote: >> > OK, OK, changed. >> > > I don't see the curlies. Did you forget to snapshot? >> > > Yup, and I just blew away my local copy after committing (Ihab said > lgtm++), so I can't. > > https://codereview.appspot.**com/8612048/<https://codereview.appspot.com/8612048/> > -- Text by me above is hereby placed in the public domain Cheers, --MarkM -- --- You received this message because you are subscribed to the Google Groups "Google Caja Discuss" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. For more options, visit https://groups.google.com/groups/opt_out.
