[PATCH 0/3] drop the SPOOLSS_BUFFER_OK macro

[David Disseldorp]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On Fri, 19 Dec 2014 18:50:57 +0100, Guenther Deschner wrote:

> Sorry David, NAK on these as well :)
> 
> The macros have been invented just for the purpose of avoiding to put
> manual checks everywhere. If you see a problem with the macros we
> should better fix them but please let's not go down the road again of
> putting all these checks back in the spoolss server explicitly.

I don't understand your objection here. This change makes the code a
whole lot easier to read, and reduces the overhead of the bounds
checking into a single comparison, rather than two or three per
operation. In doing so it only adds 43 lines of code total...

> > #define SPOOLSS_BUFFER_OK(val_true,val_false) ((r->in.offered >= 
> > *r->out.needed)?val_true:val_false)

Adding @offered and @needed arguments would make the macro less evil,
but it still wouldn't help with readability or efficiency. Please
take another look at the patches.

Cheers, David
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2

iEYEARECAAYFAlSUaXkACgkQYYkJRV19jVyXvQCfZUr1M5o0vqaTT995gAlj9fDI
W6sAoKeknj4tfqbSSNoqH5OlLDcB3J4J
=eIjQ
-----END PGP SIGNATURE-----