Relating to size of patch - I divided it maximally, Relating to subject - thank for your note - I'll take it in account next time.
-----Original Message----- From: Peter Maydell [mailto:peter.mayd...@linaro.org] Sent: Tuesday, February 26, 2013 3:54 PM To: Neiman, Anna Cc: qemu-devel@nongnu.org; Rozenman, Alex; Schreiber, Amir; Anthony Liguori; Brook, Paul Subject: Re: [PATCH] PATCH1 - infrastructure for Evaluate breakpoint condition on target.OPC_MAX_SIZE - size of translation buffer - was increased because condition translated bytecode should be executed in one block On 26 February 2013 13:50, Anna Neiman <anna_nei...@mentor.com> wrote: > > Signed-off-by: Anna Neiman <anna_nei...@mentor.com> The formatting of your patches is still a mess (huge long subject, not properly threaded with a cover letter, etc) and you're still doing too much in one patch. Hint: "add missing include guards to header files" is probably a patch of its own. -- PMM