[ITK] Word wrap for commit text at itk.org/gitweb and review.source.kitware.com?

Niels Dekker niels-xtk at xs4all.nl
Wed Feb 25 06:50:31 EST 2015


The commit log text of the commit I submitted to Gerrit yesterday had 
a paragraph of more than 280 characters, without "hard" line breaks. 
Now it appears as a single line, without any line wrapping, both at 
Gerrit, http://review.source.kitware.com/#/c/19311/ and 
http://itk.org/gitweb?p=ITK.git;a=commit;h=bbecc3b7cca9f85914235dac2d6b9e80d0d44dda
At least that's how I see it on Windows7 + Firefox 35. Which makes 
reading the text a bit uncomfortable: I'd have to use the horizontal 
scrollbar of the web browser to get to the end of the line.

Do other platforms/browsers/devices display this paragraph in a more 
convenient way, by adding word wrapping?

Otherwise, is it possible to enable word wrapping for itk.org/gitweb 
and review.source.kitware.com?

I prefer automatic word wrapping to hard line breaks, because the 
optimal position of a line break within a paragraph very much depends 
on the viewing device, the viewing software, and the preference of the 
user who is reading the text.

See also http://en.wikipedia.org/wiki/Line_wrap_and_word_wrap

BTW: Here's the log text from 
http://review.source.kitware.com/#/c/19311/ :
   ------------------------------------------------------------
The original itk_module_warnings_disable macro would remove all 
"/W[0-4]" compiler flags, but then Visual Studio (both VS2010 and 
VS2013) would automatically (!) add "/W1", causing these "D9025" 
warnings. Visual Studio does not do so when 
itk_module_warnings_disable adds "/W0" instead.

Discussed at "[ITK] ITKModuleMacros.cmake 
macro(itk_module_warnings_disable) MSVC warning D9025: overriding 
'/W1' with '/w'", 
http://public.kitware.com/pipermail/community/2015-February/008349.html
Mike Jackson confirmed the issue at 
http://public.kitware.com/pipermail/community/2015-February/008361.html
   ------------------------------------------------------------

Kind regards, Niels
-- 
Niels Dekker
Scientific programmer at LKEB, Leiden University Medical Center


More information about the Community mailing list