Dear Vikram,

thank you for your work and the patch.

Am Donnerstag, den 26.01.2012, 07:42 +0100 schrieb Vikram Narayanan:
> Vikram Narayanan ([email protected]) just uploaded a new patch set to 
> gerrit, which you can find at http://review.coreboot.org/582
> 
> -gerrit
> 
> commit 42dc925d2d86524e57ef29eef1b130e4f0e9d7a3
> Author: Vikram Narayanan <[email protected]>
> Date:   Wed Jan 25 17:44:20 2012 +0530
> 
>     dumpmmcr: fixed compilation warnings
>     
>     Fixes compilation warnings in printf

instead of almost duplicating the commit summary in the body could you
merge both and maybe paste the warning into the commit message.

    dumpmmcr: Fix compilation warnings in printf

    • cf. `man 3 printf`

You should be able to use `git commit --amend` to update the commit
message. That would be awesome.
    
>     Change-Id: Ib78937a3e1c1eecf884bde0860594cbdb574f1fe
>     Signed-off-by: Vikram Narayanan <[email protected]>
> ---
>  util/dumpmmcr/dumpmmcr.c |   10 +++++-----
>  1 files changed, 5 insertions(+), 5 deletions(-)

[…]


Thanks,

Paul

Attachment: signature.asc
Description: This is a digitally signed message part

-- 
coreboot mailing list: [email protected]
http://www.coreboot.org/mailman/listinfo/coreboot

Reply via email to