[opensource-dev] Review Request: Restructure loops that use breaks without need (reviewboard test)

Oz Linden oz at lindenlab.com
Wed Dec 1 19:57:24 PST 2010


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://codereview.secondlife.com/r/2/
-----------------------------------------------------------

Review request for Viewer.


Summary
-------

This review is mostly a first test of reviewboard.

I do have an esthetic dislike for the 'break' statement anywhere but as the end of a case, so I chose to change some instances of break usage that were not justified by any extreme need.


This addresses bug storm-606.
    http://jira.secondlife.com/browse/storm-606


Diffs
-----

  indra/newview/llappviewer.cpp bf98b026bcb1 

Diff: http://codereview.secondlife.com/r/2/diff


Testing
-------

None at all... have not even compiled it yet.


Thanks,

Oz

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.secondlife.com/pipermail/opensource-dev/attachments/20101202/94d64463/attachment.htm 


More information about the opensource-dev mailing list