struct.calcsize() incorrect

[boa124]

BPO	17617
Nosy	@mdickinson
Superseder	bpo-7355: Struct incorrectly compiles format strings

^{Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.}

Show more details

GitHub fields:

assignee = None
closed_at = <Date 2013-04-02.13:37:44.516>
created_at = <Date 2013-04-02.12:40:51.549>
labels = ['type-bug', 'library']
title = 'struct.calcsize() incorrect'
updated_at = <Date 2013-04-02.13:37:44.515>
user = 'https://bugs.python.org/boa124'

bugs.python.org fields:

activity = <Date 2013-04-02.13:37:44.515>
actor = 'mark.dickinson'
assignee = 'none'
closed = True
closed_date = <Date 2013-04-02.13:37:44.516>
closer = 'mark.dickinson'
components = ['Library (Lib)']
creation = <Date 2013-04-02.12:40:51.549>
creator = 'boa124'
dependencies = []
files = []
hgrepos = []
issue_num = 17617
keywords = []
message_count = 3.0
messages = ['185834', '185837', '185839']
nosy_count = 2.0
nosy_names = ['mark.dickinson', 'boa124']
pr_nums = []
priority = 'normal'
resolution = 'duplicate'
stage = None
status = 'closed'
superseder = '7355'
type = 'behavior'
url = 'https://bugs.python.org/issue17617'
versions = ['Python 3.4']

[boa124]

struct.calcsize('iiiii?') : 21

appears to be incorrect ; in C sizeof : 24

C code:
typedef struct {
  int	param1;
  int	param2;
  int	param3;
  int	param4;
  int	param5;
  unsigned char param6;
} struct_titi6;

main:
struct_titi6 toto3;
printf("taille toto3 : %d\n",sizeof(toto3));
printf("taille toto3.param1 : %d\n",sizeof(toto3.param1));
printf("taille toto3.param6 : %d\n",sizeof(toto3.param6));

results:
taille toto3 : 24
taille toto3.param1 : 4
taille toto3.param6 : 1

[mdickinson]

This is working as designed: the result of struct.calcsize matches the length of the string that you'll get back from struct.pack using that format. Padding isn't included after the last entry in the struct.

That's not to say that it was a good design decision, but that's the design that was chosen, and it's not realistic to change this for the bugfix releases. It may be possible to do something for Python 3.4.

You can add a '0L' to the end of your struct to force padding bytes at the end. See:

http://docs.python.org/2/library/struct.html#examples

for an example of this.

See also bpo-7189, bpo-5145.

[mdickinson]

And also bpo-7355, which led to that example being added to the documentation. Closing as duplicate.