On Sun, May 20, 2012 at 01:32:18PM +0300, Alon Levy wrote: > On Sun, May 20, 2012 at 12:03:38PM +0300, Gleb Natapov wrote: > > > > Signed-off-by: Gleb Natapov <g...@redhat.com> > > --- > > tools/acpi_extract.py | 6 +++++- > > 1 files changed, 5 insertions(+), 1 deletions(-) > > > > diff --git a/tools/acpi_extract.py b/tools/acpi_extract.py > > index 5f613e4..8038269 100755 > > --- a/tools/acpi_extract.py > > +++ b/tools/acpi_extract.py > > @@ -121,7 +121,11 @@ def aml_name_string(offset): > > if (aml[offset] != 0x08): > > die( "Name offset 0x%x: expected 0x08 actual 0x%x" % > > (offset, aml[offset])); > > - return offset + 1; > > + offset += 1 > > + # Block Name Modifier. Skip it. > > + if (aml[offset] == 0x5c or aml[offset] == 0x5e): > > You don't need parenthesis around the whole conditional. Rest of the code has it. Better to keep same style :)
> > > + offset += 1 > > + return offset; > > > > # Given data offset, find dword const offset > > def aml_data_dword_const(offset): > > -- > > 1.7.7.3 > > > > > > _______________________________________________ > > SeaBIOS mailing list > > seab...@seabios.org > > http://www.seabios.org/mailman/listinfo/seabios -- Gleb.
