[freetds] [PATCH] VMS build update

Brian Bruns brian.bruns at gmail.com
Sat Nov 6 19:05:20 EST 2004


On Sat, 6 Nov 2004 17:09:38 -0500, James K. Lowden
<jklowden at schemamania.org> wrote:
> 
> Hi Craig,
> 
> - if (freopen(options.input_filename, "r", stdin) < 0) {
> + if ((int)freopen(options.input_filename, "r", stdin) < 0) {
> 
> Could I change all these to:
> 
> if (freopen(options.input_filename, "r", stdin) < (FILE*)0) {
> 
> I think that's more honest.  BTW, does "< NULL" work better?
> 
> Similarly,
> 
> - if (options.verbose < 0) {
> + if ((int)options.verbose < 0) {
> 
> oughta be
> 
> if (options.verbose < (FILE*)0) {

Yuck, wouldn't NULL be better?  I realize NULL == 0, however it is
convention to  use NULL when dealing with pointers.

> 
> I'd prefer these unless you can convince me otherwise or your compiler
> isn't satisfied.
> 
> Thanks for the patch.
> 
> Regards,
> 
> --jkl


More information about the FreeTDS mailing list