Thank you !!
I assume you have a metadata file too that changed? That was too large to upload or something? :) Otherwise LGTM.
You are welcome :) Indeed, the metadata.cc file is not displayed only because it is too large. Cheers, Philippe. -- Philippe Liard On Mon, Jan 30, 2012 at 23:58, <lararennie@google.com> wrote: > I assume you have a metadata file too that changed? That was too large > to upload or something? :) Otherwise LGTM. > > http://codereview.appspot.com/**5600051/<http://codereview.appspot.com/5600051/> >
LGTM. Thanks for the work, Philippe!
On 2012/02/01 08:13:07, Shaopeng wrote: > LGTM. Thanks for the work, Philippe! Committed as r423 :)