home *** CD-ROM | disk | FTP | other *** search
/ NetNews Usenet Archive 1993 #3 / NN_1993_3.iso / spool / comp / sys / amiga / programm / 19191 < prev    next >
Encoding:
Internet Message Format  |  1993-01-26  |  2.0 KB
  1. Path: sparky!uunet!spool.mu.edu!yale.edu!newsserver.jvnc.net!rutgers!cbmvax!cbmehq!cbmden!hemmer!hemmer
  2. From: hemmer@hemmer.adsp.sub.org (Franz Hemmer)
  3. Newsgroups: comp.sys.amiga.programmer
  4. Subject: Re: IntuitionBase
  5. Message-ID: <hemmer.03w7@hemmer.adsp.sub.org>
  6. Date: 25 Jan 93 18:49:24 GMT
  7. References: <hemmer.03so@hemmer.adsp.sub.org> <paulk.3emk@terapin.com>
  8. Reply-To: hemmer@hemmer.adsp.sub.org (Franz Hemmer)
  9. Organization: Interactivsion
  10. Lines: 32
  11. X-NewsSoftware: GRn-beta 1.16g (11.15.92) by Michael B. Smith & Mike Schwartz
  12.  
  13. In article <paulk.3emk@terapin.com> paulk@terapin.com (Paul Kienitz) writes:
  14. > > ...which makes your sources quite difficult to read for other
  15. > > people.  You really should go for what is easiest to understand....
  16. >  
  17. > Do you honestly think "IntuitionBase=(struct IntuitionBase *)OpenLibrary
  18. > ("intuitionbase.library",37);" is easy to read?
  19.  
  20. I most certainly do if written like this:
  21.  
  22.    IntuitionBase = (struct IntuitionBase *)
  23.                    OpenLibrary ("intuition.library", 37);
  24.  
  25. Doing some defines which are named not entirely clear, and then used
  26. somewhere else in the source makes things more difficult to trace and
  27. understand. The above way is the "standard" way, and the elements used
  28. are well known and understandable to any programmer which took the time
  29. to learn a little about the include files etc. No mixing things around
  30. with defines. I have plenty experience peeking/poking around some huge
  31. commercial sources with many defines that look much like yours. This
  32. has made the sources almost impossible to understand. Though these
  33. sources now are in a well-structured form, it was a nightmare to
  34. rescue them...
  35.  
  36. I dislike these shortcuts, since they make life hard for others that
  37. follow. But hey, there's that "job security" issue again :-)
  38.  
  39. --
  40. Best regards                    Reply to: hemmer@hemmer.adsp.sub.org
  41. Franz Hemmer                    #include <stddisclaimer.h>
  42. Software Developer       _      >> Carpe Diem! - Seize the day! <<
  43.                          \\ _
  44.                  Interacti\X/ision - The name of precision
  45.