I have seen several places in the tree with chunks of code like this
(from fedex_main.c):

        fprintf(stderr,"and <object_type> is one or more of:\n");
        fprintf(stderr,"        e       entity\n");
        fprintf(stderr,"        p       procedure\n");
        fprintf(stderr,"        r       rule\n");
        fprintf(stderr,"        f       function\n");
        fprintf(stderr,"        t       type\n");
        fprintf(stderr,"        s       schema or file\n");
        fprintf(stderr,"        #       pass #\n");
        fprintf(stderr,"        E       everything (all of the above)\n");

I like to do instead this:

        fprintf(stderr,
                "and <object_type> is one or more of:\n"
                "       e       entity\n"
                "       p       procedure\n"
                "       r       rule\n"
                "       f       function\n"
                "       t       type\n"
                "       s       schema or file\n"
                "       #       pass #\n"
                "       E       everything (all of the above)\n"
                );

I believe it's easier to read (and it saves several function calls).

Is that type of change acceptable?

Regards,

-Tom

Thomas M. Browder, Jr.
Niceville, Florida
USA

------------------------------------------------------------------------------
Protect Your Site and Customers from Malware Attacks
Learn about various malware tactics and how to avoid them. Understand 
malware threats, the impact they can have on your business, and how you 
can protect your company and customers by using code signing.
http://p.sf.net/sfu/oracle-sfdevnl
_______________________________________________
BRL-CAD Developer mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/brlcad-devel

Reply via email to