groff
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Groff] Strange error messages from Groff 1.22.3


From: Eli Zaretskii
Subject: Re: [Groff] Strange error messages from Groff 1.22.3
Date: Sat, 22 Nov 2014 12:37:20 +0200

> Date: Thu, 20 Nov 2014 19:47:19 +0100 (CET)
> Cc: address@hidden
> From: Werner LEMBERG <address@hidden>
> 
> >> Sorry for the delayed answer.  Your patch looks fine, and I'm going
> >> to apply it soon.  Thanks a lot.
> > 
> > Great, thanks.  Now I can publish the binary with that patch and
> > claim that it was accepted upstream.
> 
> Now committed, with minor fixes.

Out of curiosity, why did you prefer this:

  +#if defined(__MSDOS__) || (defined(_WIN32) && !defined(__CYGWIN__))
  +void normalize_for_lf (string &fn)
  +{
  +  int fnlen = fn.length();
  +  for (int i = 0; i < fnlen; i++) {
  +    if (fn[i] == '\\')
  +      fn[i] = '/';
  +  }
  +}
  +#else
  +void normalize_for_lf (string &)
  +{
  +}
  +#endif

to my original change:

  +void normalize_for_lf (string &fn)
  +{
  +#if defined(__MSDOS__) || (defined(_WIN32) && !defined(__CYGWIN__))
  +  int fnlen = fn.length();
  +  for (int i = 0; i < fnlen; i++)
  +    {
  +      if (fn[i] == '\\')
  +       fn[i] = '/';
  +    }
  +#endif
  +}



reply via email to

[Prev in Thread] Current Thread [Next in Thread]