On Mon, Nov 29, 2010 at 03:02, Brian Norris wrote: > + errmsg_die("Can't specify a negative blockalign '%d'", > + blockalign); personally, i'd avoid quotes on non-string types. maybe also mention the command line option that is "blockalign" ? otherwise, these changes all look fine to me -mike