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

[Andreas Cadhalpun]

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

-------------- next part --------------
A non-text attachment was scrubbed...
Name: improve-expr_int_format-documentation.patch
Type: text/x-diff
Size: 1428 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140907/762dad22/attachment.bin>