On 5 October 2015 at 19:11, Eduardo Habkost <ehabk...@redhat.com> wrote:
> On Sun, Oct 04, 2015 at 07:44:29PM +0100, Peter Maydell wrote:
>> On 17 September 2015 at 17:32, Paolo Bonzini <pbonz...@redhat.com> wrote:
>> >
>> >
>> > On 17/09/2015 18:16, Peter Maydell wrote:
>> >> On 17 September 2015 at 17:00, Paolo Bonzini <pbonz...@redhat.com> wrote:
>> >>>
>> >>>
>> >>> On 17/09/2015 16:24, Peter Maydell wrote:
>> >>>> Can we revert this one, please? Checkpatch now warns about constructs
>> >>>> like
>> >>>> typedef struct MyDevice {
>> >>>> DeviceState parent;
>> >>>>
>> >>>> int reg0, reg1, reg2;
>> >>>> } MyDevice;
>>
>> > I think it varies depending on the maintainer. PPC, USB, SCSI, ACPI all
>> > use a separate typedef. I'll prepare a revert.
>>
>> Ping on that revert patch? I can't find it onlist...
>
> The x86 pull request I sent today triggers this error, BTW. I hope it
> won't make the pull request be automatically rejected.
Nope, because I don't run checkpatch on pull requests.
-- PMM
