[opensource-dev] UI produces just grey boxes

2011-04-04 Thread Glimmering Sands
Dear Developers, Since November last I have been unable to build a a version of the viewer that does not cause the UI to suffer from what I might call severe grey fatigue syndrome. Please see the attached screen shots to understand what I mean. I am building using Xcode Version 3.2.3. I have pu

Re: [opensource-dev] Snowstorm PO review build

2011-04-04 Thread Anya Kanevsky
all approved! thank you for testing help. a 2011/3/31 Oz Linden (Scott Lawrence) > > http://automated-builds-secondlife-com.s3.amazonaws.com/hg/repo/oz_viewer-poreview/rev/225458/index.html > > Here are the issues addressed in that build: > > STORM-399

Re: [opensource-dev] autobuild & VS 2010 support merged to viewer-development

2011-04-04 Thread Trilo Byte
Last build with working multi-wearables (on Mac client, anyways) was 225589. I've created a JIRA issue for the bug - https://jira.secondlife.com/browse/VWR-25421 If any Windows/Linux users could please check this out and see if they can reproduce, I'd appreciate it. TriloByte Zanzibar On Apr

Re: [opensource-dev] autobuild & VS 2010 support merged to viewer-development

2011-04-04 Thread Trilo Byte
Found something that appears to have broken in 225670 - multi-wearables. Adding multiple attachments still works as expected, right click on an attachment in your inventory or the My Appearance tab and choose Add to add it to your currently worn outfit. However, multiple layers is broken - the

Re: [opensource-dev] Review Request: Squared all dist_vec() based comparisons and other dist_vec() operations where sensible.

2011-04-04 Thread Ricky
And since that failed too, I'll try the tried-and-true email list! Ha! In llselectmgr.cpp (around lines 6586-6595) we've been beating our heads against some rather... interesting... code. I had tried adding a variable to make things clearer, but I now suggest to jump sideways, and try another rou

Re: [opensource-dev] Review trouble (was: Review Request: Squared all dist_vec() based comparisons and other dist_vec() operations where sensible.)

2011-04-04 Thread Ricky
I was just doing a straight review, not a source review, and all the text was gone. Then I added a comment to my own empty review, and poof, all but the first line are gone. Very strange. I did paste in some code as part of an idea I was trying to talk about, but that started on line 3 of the res

[opensource-dev] Review trouble (was: Review Request: Squared all dist_vec() based comparisons and other dist_vec() operations where sensible.)

2011-04-04 Thread Boroondas Gupte
On 04/05/2011 01:49 AM, Ricky wrote: > ok, this is getting annoying. Any idea why I am having issues posting > to the review? Mayhaps you added comments to code-lines but didn't save them? Also, comments with overlapping code-line ranges (e.g. one comment about lines 1-7 and one about lines 3-9 of

Re: [opensource-dev] Review Request: Squared all dist_vec() based comparisons and other dist_vec() operations where sensible.

2011-04-04 Thread Ricky
ok, this is getting annoying. Any idea why I am having issues posting to the review? Ricky Cron Stardust On Mon, Apr 4, 2011 at 4:48 PM, Cron Stardust wrote: >This is an automatically generated e-mail. To reply, visit: > http://codereview.secondlife.com/r/199/ > > Huh.. long winded speech

Re: [opensource-dev] Review Request: Squared all dist_vec() based comparisons and other dist_vec() operations where sensible.

2011-04-04 Thread Cron Stardust
> On April 4, 2011, 4:39 p.m., Cron Stardust wrote: > > Huh.. long winded speech became dust on pressing publish... Here goes again: - Cron --- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.co

Re: [opensource-dev] Review Request: Squared all dist_vec() based comparisons and other dist_vec() operations where sensible.

2011-04-04 Thread Cron Stardust
--- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/199/#review546 --- - Cron On April 4, 2011, 10:34 a.m., Cron Stardust wrote: > > -

[opensource-dev] Review Request: (STORM-941) IM log naming should go by SL name, not DN.

2011-04-04 Thread Seth ProductEngine
--- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/249/ --- Review request for Viewer. Summary --- Fixed IM history to use the res

Re: [opensource-dev] Review Request: Squared all dist_vec() based comparisons and other dist_vec() operations where sensible.

2011-04-04 Thread Cron Stardust
> On April 4, 2011, 2:33 p.m., Boroondas Gupte wrote: > > indra/newview/llselectmgr.cpp, lines 6591-6592 > > > > > > This comment only really makes sense to those aware of this change. > > Those reading the code la

Re: [opensource-dev] Review Request: Squared all dist_vec() based comparisons and other dist_vec() operations where sensible.

2011-04-04 Thread Boroondas Gupte
--- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/199/#review544 --- indra/newview/llselectmgr.cpp

Re: [opensource-dev] Review Request: Squared all dist_vec() based comparisons and other dist_vec() operations where sensible.

2011-04-04 Thread Cron Stardust
--- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/199/ --- (Updated April 4, 2011, 10:34 a.m.) Review request for Viewer. Changes --

Re: [opensource-dev] Review Request: Squared all dist_vec() based comparisons and other dist_vec() operations where sensible.

2011-04-04 Thread Oz Linden
> On March 13, 2011, 5:34 a.m., Boroondas Gupte wrote: > > indra/newview/llfloaterchat.cpp, lines 416-419 > > > > > > If storing return values of functions (or inline computations), just so > > we don't have to call t

Re: [opensource-dev] autobuild & VS 2010 support merged to viewer-development

2011-04-04 Thread Opensource Obscure
I ran both last vs2010 and v-d builds for many hours in the last days, trying to use as much SL features as possible. Performances and behaviour were quite satisfying. I had no issues, except: - unusual high inventory loading time in BlueSteel Sandbox 1 (see my report to server-beta mailing list)

Re: [opensource-dev] autobuild & VS 2010 support merged to viewer-development

2011-04-04 Thread Mike Chase
On Sun, 2011-04-03 at 10:57 -0400, Oz Linden (Scott Lawrence) wrote: > On 2011-04-03 9:27, Mike Chase wrote: > > Quick pass through on Linux and all seemed ok. Profiles worked fine and > > the annoying pause that previouly made the viewer unusable seems fixed. > > Glad to hear that the proble

Re: [opensource-dev] autobuild & VS 2010 support merged to viewer-development

2011-04-04 Thread Nicky Perian
Second Life 2.6.3 (225584) Apr 1 2011 02:02:52 (Project Viewer - VS2010) Release Notes You are at 255,034.0, 256,238.0, 28.7 in Gibson located at sim7019.aditi.lindenlab.com (216.82.40.115:13001) DRTSIM-42 11.03.29.225233 Release Notes CPU: Intel(R) Core(TM) i5 CPU 650 @ 3.20GHz (3192.

Re: [opensource-dev] Review Request: STORM-1095 Chat preferences > font size should increase size of input text in the chat box

2011-04-04 Thread Jonathan Yap
> On April 3, 2011, 8:46 p.m., Merov Linden wrote: > > indra/newview/llviewerchat.cpp, lines 38-39 > > > > > > Should be initialized to NULL. I fixed that initialization to NULL issue and that change was in the last PO

Re: [opensource-dev] Review Request: STORM-1095 Chat preferences > font size should increase size of input text in the chat box

2011-04-04 Thread Jonathan Yap
> On March 30, 2011, 3:16 p.m., Boroondas Gupte wrote: > > indra/llui/lllineeditor.cpp, lines 2294-2297 > > > > > > ... this simple setter isn't implemented right at the declaration (i.e. > > in indra/llui/lllineedit