On Tue, May 12, 2015 at 07:42:17PM +0200, Markus Armbruster wrote: > Eduardo Habkost <ehabk...@redhat.com> writes: > > On Tue, May 12, 2015 at 05:38:37PM +0200, Markus Armbruster wrote: > > [...] > >> > @@ -699,8 +701,9 @@ > >> > # data is sent to the client, the guest may no longer be halted. > >> > ## > >> > { 'struct': 'CpuInfo', > >> > - 'data': {'CPU': 'int', 'current': 'bool', 'halted': 'bool', '*pc': > >> > 'int', > >> > - '*nip': 'int', '*npc': 'int', '*PC': 'int', 'thread_id': 'int'} > >> > } > >> > + 'data': {'CPU': 'int', 'current': 'bool', 'halted': 'bool', > >> > qom_path': 'str', > >> > >> Long line. > > > > It has exactly 80 characters. > > Several characters too wide for my taste.
I thought I had broken some documented line length limit. > > I just realized checkpatch is fine with 80. I'll revert my line wrap if > you feel strongly about it. Please keep your line wrap, I don't mind. -- Eduardo