[opensource-dev] Review Request: STORM-826 (partial): fix line endings in files that use a mix of CRLF and LF

Oz Linden oz at lindenlab.com
Thu Jan 6 10:20:31 PST 2011


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

(Updated Jan. 6, 2011, 10:20 a.m.)


Review request for Viewer.


Summary (updated)
-------

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.

The diff apparently won't show the change in line ending characters.... see the repo (in the issue) for the real change.
I expanded the scope of this to also convert some files that were all the same CRLF endings but did not obviously need to be.   I left files that were clearly windows specific alone on the theory that non-windows users probably won't need to touch them, and the windows tools might care.


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


Diffs (updated)
-----

  indra/cmake/GetPrerequisites_2_8.cmake 6d44f0d85a80 
  indra/cmake/LLAddBuildTest.cmake 6d44f0d85a80 
  indra/newview/llfloaterwebcontent.h 6d44f0d85a80 
  indra/newview/llfloaterwebcontent.cpp 6d44f0d85a80 
  indra/newview/llimview.h 6d44f0d85a80 
  indra/newview/llimview.cpp 6d44f0d85a80 
  indra/newview/lllogchat.cpp 6d44f0d85a80 
  indra/newview/tests/llremoteparcelrequest_test.cpp 6d44f0d85a80 
  indra/viewer_components/updater/tests/llupdaterservice_test.cpp 6d44f0d85a80 

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


Testing
-------


Thanks,

Oz

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.secondlife.com/pipermail/opensource-dev/attachments/20110106/327e4721/attachment.htm 


More information about the opensource-dev mailing list