[FFmpeg-devel] [PATCH 06/13] MIPS i6400 core do has an FPU, so not disable it

Ronald S. Bultje rsbultje at gmail.com
Tue Nov 17 18:28:08 CET 2015


Hi,

On Tue, Nov 17, 2015 at 11:40 AM, Vicente Olivert Riera <
Vincent.Riera at imgtec.com> wrote:

> Dear Moritz Barsnick,
>
> On 11/17/2015 04:20 PM, Moritz Barsnick wrote:
> > On Tue, Nov 17, 2015 at 15:07:56 +0000, Vicente Olivert Riera wrote:
> >> file. Do you mean the patch subject instead of patch name? If so, what's
> >> the error?
> >
> > Indeed, the grammar:
> >
> >> [PATCH 06/13] MIPS i6400 core do has an FPU, so not disable it
> >
> > -> MIPS i6400 core has an FPU, so not disable it
> > or with emphasis
> > -> MIPS i6400 core does have an FPU, so not disable it
> > (infinitive of "to have" following the third person singular of the
> > helper verb "to do")
>
> That's right, thanks for the clarification. Does that mistake require
> the patch to be resent?


You can wait for other comments, i.e. you don't have to immediately do it
right now, but I would suggest to resend it, yes.

The risk of not doing so is that someone (e.g. me) scans over the
mailinglist for approved patches to mass-apply all of them at once, and
then I'll probably forget to amend the subject of this individual patch.

Ronald


More information about the ffmpeg-devel mailing list