[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