On Thu, Jul 24, 2014 at 04:32:29PM +0400, Maria Kustova wrote: > ['>I', offset, 0x6803f857, 'ext_magic'], > - ['>I', offset + UINT32_S, len(feature_tables)*48, > 'ext_length'] > + # Length of one feature table equals to 3 fields and 48 bytes > + ['>I', offset + UINT32_S, len(feature_tables)/3*48,
Please use spaces around operators in arithmetic expressions: len(feature_tables) / 3 * 48 There are other instances of missing space in this patch.
pgpcik6I7hbEf.pgp
Description: PGP signature