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.

Attachment: pgpcik6I7hbEf.pgp
Description: PGP signature

Reply via email to