Re: [Qemu-devel] [PATCH 4/5] bios-tables-test: fix ASL normalization false positive

2014-07-29 Thread Laszlo Ersek
On 07/28/14 17:34, Paolo Bonzini wrote: > My version of IASL (from RHEL7) puts two newlines between the head comment > and the DefinitionBlock property. Kill all newlines after the comment, > so that normalize_asl works properly. > > Signed-off-by: Paolo Bonzini > --- > tests/bios-tables-test.c

[Qemu-devel] [PATCH 4/5] bios-tables-test: fix ASL normalization false positive

2014-07-28 Thread Paolo Bonzini
My version of IASL (from RHEL7) puts two newlines between the head comment and the DefinitionBlock property. Kill all newlines after the comment, so that normalize_asl works properly. Signed-off-by: Paolo Bonzini --- tests/bios-tables-test.c | 6 +- 1 file changed, 5 insertions(+), 1 deleti

[Qemu-devel] [PATCH 4/5] bios-tables-test: fix ASL normalization false positive

2014-07-28 Thread Paolo Bonzini
My version of IASL (from RHEL7) puts two newlines between the head comment and the DefinitionBlock property. One was already removed because the test uses sizeof instead of strlen, but the extra one breaks the detection of DefinitionBlock. Killing all newlines after the comment drops the warning.