Lars Gullik Bjønnes wrote:

> Angus Leeming <[EMAIL PROTECTED]> writes:
> 
> | @@ -217,8 +217,19 @@ bool setLyxPaths()
> |  bool followlink;
> |  do {
> |  // Path of binary/../share/name of binary/
> | -           searchpath += NormalizePath(AddPath(binpath, "../share/") +
> | -                 OnlyFilename(binname)) + ';';
> | +
> | +           string const exe_name = OnlyFilename(binname);
> | +           string const lyx_system_dir_name =
> | +#ifdef _WIN32
> | +                   suffixIs(exe_name, ".exe") ?
> | +                           ChangeExtension(exe_name, "") :
> | +                           exe_name;
> | +#else
> | +                   exe_name;
> | +#endif
> 
> Can we please not split statements with ifdef?

Sure, but that is just more 'whitespace commentary'. Are you in favour of 
the substance of the proposal or should we take this back to the drawing 
board?

-- 
Angus

Reply via email to