Am 11.02.2012 10:19, schrieb Blue Swirl:
> On Wed, Feb 8, 2012 at 15:36, Andreas Färber wrote:
>> This is not about whether or not we put a space somewhere.
>>
>> It's about reviewers and SubmitAPatch telling people to run
>> checkpatch.pl on patches and checkpatch.pl reporting this as an ERROR,
>
On Wed, Feb 8, 2012 at 15:36, Andreas Färber wrote:
> Am 08.02.2012 16:23, schrieb Anthony Liguori:
>> On 02/08/2012 09:04 AM, malc wrote:
>>> On Wed, 8 Feb 2012, Andreas F?rber wrote:
>>>
Arbitrarily reformatting your files is not okay. If you want a different
formatting, you need to fi
On Wed, Feb 8, 2012 at 15:23, Anthony Liguori wrote:
> On 02/08/2012 09:04 AM, malc wrote:
>>
>> On Wed, 8 Feb 2012, Andreas F?rber wrote:
>>
>>> malc,
>>>
>>> Arbitrarily reformatting your files is not okay. If you want a different
>>> formatting, you need to fix checkpatch.pl first to not error
On 02/09/2012 03:48 AM, Markus Armbruster wrote:
Anthony Liguori writes:
On 02/08/2012 09:04 AM, malc wrote:
On Wed, 8 Feb 2012, Andreas F?rber wrote:
malc,
Arbitrarily reformatting your files is not okay. If you want a different
formatting, you need to fix checkpatch.pl first to not error
Anthony Liguori writes:
> On 02/08/2012 09:04 AM, malc wrote:
>> On Wed, 8 Feb 2012, Andreas F?rber wrote:
>>
>>> malc,
>>>
>>> Arbitrarily reformatting your files is not okay. If you want a different
>>> formatting, you need to fix checkpatch.pl first to not error on that
>>> formatting in your
On 02/08/2012 09:36 AM, Andreas Färber wrote:
Am 08.02.2012 16:23, schrieb Anthony Liguori:
On 02/08/2012 09:04 AM, malc wrote:
On Wed, 8 Feb 2012, Andreas F?rber wrote:
Arbitrarily reformatting your files is not okay. If you want a different
formatting, you need to fix checkpatch.pl first to
Am 08.02.2012 16:23, schrieb Anthony Liguori:
> On 02/08/2012 09:04 AM, malc wrote:
>> On Wed, 8 Feb 2012, Andreas F?rber wrote:
>>
>>> Arbitrarily reformatting your files is not okay. If you want a different
>>> formatting, you need to fix checkpatch.pl first to not error on that
>>> formatting in
Am 08.02.2012 16:04, schrieb malc:
> On Wed, 8 Feb 2012, Andreas F?rber wrote:
>
>> Arbitrarily reformatting your files is not okay. If you want a different
>> formatting, you need to fix checkpatch.pl first to not error on that
>> formatting in your files.
>
> It was always formatter like this (
On 02/08/2012 09:04 AM, malc wrote:
On Wed, 8 Feb 2012, Andreas F?rber wrote:
malc,
Arbitrarily reformatting your files is not okay. If you want a different
formatting, you need to fix checkpatch.pl first to not error on that
formatting in your files.
It was always formatter like this (inter
On Wed, 8 Feb 2012, Andreas F?rber wrote:
> malc,
>
> Arbitrarily reformatting your files is not okay. If you want a different
> formatting, you need to fix checkpatch.pl first to not error on that
> formatting in your files.
It was always formatter like this (internally consistent), then others
malc,
Arbitrarily reformatting your files is not okay. If you want a different
formatting, you need to fix checkpatch.pl first to not error on that
formatting in your files.
And please post your patches to the mailing list.
Thanks,
Andreas
Am 08.02.2012 11:11, schrieb Evgeny Voevodin:
> The las
The last commit "Restore consistent formatting"
(cf4dc461a4cfc3e056ee24edb26154f4d34a6278) which is on the master but
absent in mailing list results in such output of checkpatch.pl:
./scripts/checkpatch.pl --no-tree --file hw/cs4231a.c
...
total: 107 errors, 88 warnings, 696 lines checked
12 matches
Mail list logo
