Code review comment for lp://staging/~michihenning/thumbnailer/no-tmp-file

Revision history for this message
James Henstridge (jamesh) wrote :

I think it'd be better to remove the annotation in that case. "ay" literally means "array of bytes", so we're not really adding any semantic information.

The annotation for the QSize arguments is there because this isn't the default marshalling: a struct of two integers could be anything, so in this case the annotation is actually required and provides new information.

« Back to merge proposal