qemu-devel
[Top][All Lists]
Advanced

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

Re: [Qemu-devel] [Qemu-trivial] [PATCH v3] misc: Spelling and grammar fi


From: Michael Tokarev
Subject: Re: [Qemu-devel] [Qemu-trivial] [PATCH v3] misc: Spelling and grammar fixes in comments
Date: Mon, 21 Oct 2013 12:46:28 +0400
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130922 Icedove/17.0.9

So guys, do we agree to apply this or not, after all? :)

Thanks,

/mjt

12.10.2013 00:28, Don Koch wrote:
On 10/11/2013 04:10 PM, Peter Maydell wrote:
On 12 October 2013 05:03, Don Koch <address@hidden> wrote:
On 10/11/2013 03:34 PM, Stefan Weil wrote:
* it's -> its
* grammar fix in ui/vnc-enc-zywrle.h

Signed-off-by: Stefan Weil <address@hidden>
---

This is v3 of http://patchwork.ozlabs.org/patch/257416/.

v3:
Don't try to improve a strange sentence in the same context ("Of cause [...]").

   In this method, H/L and X0/X1 is always same position.
- This lead us to more speed and less memory.
+ This leads us to more speed and less memory.
   Of cause, the result of both method is quite same
   because it's only difference that coefficient position.
  */


Since you're in the neighborhood, this paragraph needs more fixes:

No, see the note above and the previous discussion.

In that case, there in no point in fixing just the lead/leads typo.

  In this method, H/L and X0/X1 is always in the same position.
  This leads us to more speed and less memory.
  Of course, the result of both methods is the same
  because the only difference is the coefficient position.

Unless you want to say you've reviewed the code and
are completely confident that the original author meant
"the same" or "exactly the same" and not "quite similar"
then we need to leave the original wording alone.

-- PMM


-d







reply via email to

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