----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/70/#review115 -----------------------------------------------------------
"View Diff" doesn't display anything and with "Download Diff", I get a patch file that doesn't change anything: It removes some lines and adds back the exact same lines, with the same line endings (just LF). The surrounding lines (in the diff context) seem also to be LF-ended, even though one of the actual original files (indra/newview/llfloaterwebcontent.h) has CRLF endings, except for the line touched by the patch. I don't know how much of these are review-board and/or diffing issues. Please point us to a hg changeset so we can review the actual change. - Boroondas On Jan. 5, 2011, 7:19 a.m., Oz Linden wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://codereview.secondlife.com/r/70/ > ----------------------------------------------------------- > > (Updated Jan. 5, 2011, 7:19 a.m.) > > > Review request for Viewer. > > > Summary > ------- > > This is a simple change to correct existing line endings - I scanned all of > viewer-development to identify files that had a mixture of CRLF and LF > endings and converted them to just LF. > > What to do about preventing future such will be dealt with separately... > > > This addresses bug storm-826. > http://jira.secondlife.com/browse/storm-826 > > > Diffs > ----- > > indra/newview/llfloaterwebcontent.h 845cab866155 > indra/newview/llimview.h 845cab866155 > indra/newview/llimview.cpp 845cab866155 > indra/newview/lllogchat.cpp 845cab866155 > > Diff: http://codereview.secondlife.com/r/70/diff > > > Testing > ------- > > > Thanks, > > Oz > >
_______________________________________________ Policies and (un)subscribe information available here: http://wiki.secondlife.com/wiki/OpenSource-Dev Please read the policies before posting to keep unmoderated posting privileges
