[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-trivial] [PATCH] checkpatch: Check *.cc files and allow C99 co
Re: [Qemu-trivial] [PATCH] checkpatch: Check *.cc files and allow C99 comments for C++ code
Sat, 07 Jun 2014 18:10:08 +0200
Mozilla/5.0 (X11; Linux i686; rv:24.0) Gecko/20100101 Thunderbird/24.5.0
Am 07.06.2014 17:58, schrieb Peter Maydell:
> On 7 June 2014 16:42, Stefan Weil <address@hidden> wrote:
>> QEMU uses both *.cc and *.cpp for some files with C++ code:
>> *.cpp files were already checked. Add the cc pattern, so *.cc are checked
>> now, too.
> Note that there's not much point in checking libvixl's .cc files,
> because we're only going to change these for (a) updates
> to new upstream versions and (b) fixes for serious issues
> that can't wait for a new upstream version; they're never
> going to match our coding style.
> -- PMM
disas/arm-a64.cc is purely QEMU code and should use the QEMU coding
style. For Linux header files, libvixl and other imported code, people
usually know how they have to handle them. My own patch also fails when
it is checked with checkpatch.pl: it uses tabs, and some lines are too
long. It's also imported code...