Re: [PATCH] po/de.po: Fix possible typo which makes gettext-0.22 unhappy

[Dave Chinner <david@fromorbit.com>]

On Wed, Jun 21, 2023 at 01:56:08PM +0200, Lars Wendler wrote:
> Am Wed, 21 Jun 2023 13:29:03 +0200
> schrieb Holger Hoffstätte <holger@applied-asynchrony.com>:
> 
> > On 2023-06-21 12:55, Lars Wendler wrote:
> > > diff --git a/po/de.po b/po/de.po
> > > index 944b0e91..a6f8fde1 100644
> > > --- a/po/de.po
> > > +++ b/po/de.po
> > > @@ -3084,7 +3084,7 @@ msgstr "%llu Spezialdateien\n"
> > >   #: .././estimate/xfs_estimate.c:191
> > >   #, c-format
> > >   msgid "%s will take about %.1f megabytes\n"
> > > -msgstr "%s wird etwa %.lf Megabytes einnehmen\n"
> > > +msgstr "%s wird etwa %.1f Megabytes einnehmen\n"
> > 
> > I don't see the difference..?
> > Both the added and removed line are the same.
> > 
> > -h
> 
> I suppose depending on the font, it's quite hard to distinguish the two
> lines.

I didn't see it, and the commit message doesn't explain anything,
either. Pointing to a bugzilla somewhere does not make a valid
commit message....

> The removed line contains "%.lf" with a lowercase letter L.
> The added line contains "%.1f" where the lowercase letter L was replaced
> with the digit 1.

... whereas this explains what the bug being fixed is, and allows the
reviewer to see the subtle change being made. i.e. you just wrote
the commit message that should have been in the patch in the first
place. :)

Can you please resend the fix with the commit message updated?

Cheers,

Dave.
-- 
Dave Chinner
david@fromorbit.com