Re: [opensource-dev] Review Request: Enable legacy viewer C++ tests in indra/test.

2011-10-05 Thread Nat Goodspeed
> On Oct. 5, 2011, 10:42 a.m., Nat Goodspeed wrote: > > indra/test/llsdmessagebuilder_tut.cpp, line 87 > > > > > > Srsly?? I'd much rather see the definition of _PREHASH_Test0 et al. > > change to make this work. Are t

Re: [opensource-dev] Review Request: Enable legacy viewer C++ tests in indra/test.

2011-10-05 Thread Log Linden
> On Oct. 5, 2011, 10:42 a.m., Nat Goodspeed wrote: > > indra/test/llsd_new_tut.cpp, line 762 > > > > > > Hmm, where did this code come from? Server side? Yep, that version of this test seemed to be newer and fixed so

Re: [opensource-dev] Review Request: Enable legacy viewer C++ tests in indra/test.

2011-10-05 Thread Nat Goodspeed
--- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/479/#review1043 --- indra/test/llsd_new_tut.cpp

Re: [opensource-dev] Review Request: Enable legacy viewer C++ tests in indra/test.

2011-10-04 Thread Log Linden
--- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/479/ --- (Updated Oct. 4, 2011, 10:23 a.m.) Review request for Viewer and Nat Goodsp

Re: [opensource-dev] Review Request: Enable legacy viewer C++ tests in indra/test.

2011-09-28 Thread Vadim ProductEngine
--- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/479/#review1039 --- Ship it! - Vadim On Sept. 19, 2011, 12:23 p.m., Log Linden wro

[opensource-dev] Review Request: Enable legacy viewer C++ tests in indra/test.

2011-09-19 Thread Log Linden
--- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/479/ --- Review request for Viewer and Nat Goodspeed. Summary --- As part of th