Garrett D'Amore wrote: > Ian Collins wrote: > > Roland Mainz wrote: > >> Ian Collins wrote: > >> > >>> Consider something like > >>> > >>> char buffer[max_consumer_size+string_length]; > >>> > >>> Give the constant for 140 a descriptive and and save yourself a comment. > >>> > >> Maybe a #define is better here since some compilers add an extra symbol > >> which is IMO not worth the space in the resulting binary... > >> > > Unlikely unless you take the address of the constant. > > > > Even if it did, I'm sure an extra 4 or 8 bytes won't break the bank. > > (Almost) anything's better than a #define! > > Plus it can then be made patchable, which you can't do (as easily) with > a #define.
Erm, how should this be "patchable" ? You can't change it via a debugger (assuming the storage is on a read-only page) and Solaris doesn't ship binary patches (e.g. "patchadd" and "patchrm" operate on whole files, not binary diffs). ---- Bye, Roland -- __ . . __ (o.\ \/ /.o) [EMAIL PROTECTED] \__\/\/__/ MPEG specialist, C&&JAVA&&Sun&&Unix programmer /O /==\ O\ TEL +49 641 7950090 (;O/ \/ \O;) _______________________________________________ opensolaris-code mailing list [email protected] http://mail.opensolaris.org/mailman/listinfo/opensolaris-code
