[FFmpeg-devel] [PATCH] fix two spelling errors

Michael Niedermayer michaelni at gmx.at
Sun Sep 7 18:35:42 CEST 2014


On Sun, Sep 07, 2014 at 12:08:18AM +0200, Andreas Cadhalpun wrote:
> On 06.09.2014 13:45, Reimar Döffinger wrote:
> >On Sat, Sep 06, 2014 at 12:13:30PM +0200, Andreas Cadhalpun wrote:
> >>-Effectively this allows to add padding with zeros from the left.
> >>+Effectively this allows one to add padding with zeros from the left.
> >
> >Huh? I don't think the second is more correct, and it sure sounds worse.
> >What is this trying to say anyway?
> 
> The 'allows to' phrase is detected as spelling mistake by lintian.
> But I agree, that it doesn't sound much better with the 'one' added.
> 
> >Does it mean to say "This will pad with zeros from the left"?
> >The words "+Effectively" and "allows" to me seem to be just useless
> >and confusing fillers and should go, most uses of such filler words
> >is misplaced in documentation.
> 
> I'm wondering whether it will only pad, or also truncate the output,
> if it is too long? (I think it won't.)
> Reading the whole paragraph again, I noticed that some articles are
> missing, as well.
> Attached is a patch trying to improve the wording of this paragraph.
> 
> Best regards,
> Andreas
> 

>  filters.texi |   10 +++++-----
>  1 file changed, 5 insertions(+), 5 deletions(-)
> ad05573ca43155e4e45701dbb491f30b7eb02c98  improve-expr_int_format-documentation.patch
> From be0713f01537927ff4f87ec7f6ecb114beed2fc6 Mon Sep 17 00:00:00 2001
> From: Andreas Cadhalpun <Andreas.Cadhalpun at googlemail.com>
> Date: Sat, 6 Sep 2014 23:49:43 +0200
> Subject: [PATCH] doc/filters.texi: improve the wording of the expr_int_format
>  documentation

applied

thanks

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Opposition brings concord. Out of discord comes the fairest harmony.
-- Heraclitus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140907/e222bfe8/attachment.asc>


More information about the ffmpeg-devel mailing list