Am 07.01.2015 um 21:26 schrieb Jean-Marc Lasgouttes:
Le 07/01/2015 02:33, Uwe Stöhr a écrit :
compiling today's branch I get this compilation warning:
..\..\src\Paragraph.cpp(3869): warning C4018: '<': conflict between
'signed' and 'unsigned' [D:\LyXGit\2.1.x\compile-result\src\LyX.vcxproj]
Are you sure of the line number?
Yes, this line reads
if (pos < from + lyxrc.completion_minlength)
Note that this is branch-only. In master Georg added this comment:
// Work around MSVC warning: The statement
// if (pos < from + lyxrc.completion_minlength)
// triggers a signed vs. unsigned warning.
// I don't know why this happens, it could be a MSVC bug, or
// related to LLP64 (windows) vs. LP64 (unix) programming
// model, or the C++ standard might be ambigous in the section
// defining the "usual arithmetic conversions". However, using
// a temporary variable is safe and works on all compilers.
For safety reason this workaround should also be used in branch, right?
regards Uwe
