home *** CD-ROM | disk | FTP | other *** search
/ NetNews Usenet Archive 1992 #26 / NN_1992_26.iso / spool / comp / sys / mac / programm / 18288 < prev    next >
Encoding:
Internet Message Format  |  1992-11-11  |  2.5 KB
  1. Path: sparky!uunet!pageworks.com!world!eff!sol.ctr.columbia.edu!zaphod.mps.ohio-state.edu!pacific.mps.ohio-state.edu!linac!att!ucbvax!ucdavis!franklin
  2. From: franklin@endive.eecs.ucdavis.edu (Paul Franklin)
  3. Newsgroups: comp.sys.mac.programmer
  4. Subject: Re: INIT's/System Extension Questions
  5. Message-ID: <FRANKLIN.92Nov11154658@endive.eecs.ucdavis.edu>
  6. Date: 11 Nov 92 23:46:58 GMT
  7. References: <1992Nov5.184623.15326@asuvax.eas.asu.edu> <19028@ucdavis.ucdavis.edu>
  8.     <1992Nov9.142944.3187@dartvax.dartmouth.edu>
  9. Sender: usenet@ucdavis.ucdavis.edu
  10. Organization: University of California at Davis
  11. Lines: 39
  12. In-reply-to: Mark.R.Valence@dartmouth.edu's message of 9 Nov 92 14:29:44 GMT
  13.  
  14. In article <1992Nov9.142944.3187@dartvax.dartmouth.edu> Mark.R.Valence@dartmouth.edu (Mark R. Valence) writes:
  15. >In article <19028@ucdavis.ucdavis.edu>
  16. >cklarson@taquito.engr.ucdavis.edu (Christopher Klaus Larson) writes:
  17. >
  18. >> Feel free to use this as you see fit. Optimizations and bug fixes welcome.
  19. >
  20. >I have two minor comments:
  21. >
  22. >1.  You save A5, and then restore A5 AND CurrentA5 to the A5's original
  23. >value.  This is dangerous, as you are assuming A5 points to a valid
  24. >quickdraw globals area upon entry to your code.  I would suggest
  25. >instead that you save BOTH A5 and CurrentA5 in separate locations, and
  26. >restore them separately.  If you don't, you will see some weird bugs
  27. >with some pretty widely used INITs (I won't mention any names).
  28.  
  29. Very true. I do assume that A5 and CurrentA5 contain the same value
  30. upon entry -- a POOR assumption. The value of CurrentA5 should be
  31. saved and restored to its value on entry, not restored to A5's entry
  32. value.
  33.  
  34. >2.  You should probably give some credit to the guys who wrote the
  35. >original and most widely used version of this code.  They were the
  36. >ones, I believe, who first used CurApName for temporary storage, and
  37. >made it the "standard".  I, too have written my own version of this
  38. >code (it's smaller than the original, and does the same stuff), but my
  39. >code still credits the original source.
  40.  
  41. Absolutely. The original routine was written by Paul Mercer, Darin Adler,
  42. and Paul Snively from an idea by Steve Capps. Their code did standardize
  43. the use of CurApName for temporary storage. In no way do I attempt to
  44. take credit for thier work and I apologize for this omission.
  45.  
  46. Thanks for the comments.
  47.  
  48. (Sorry it took so long for me to respond -- we have had some problems
  49. with our news server so I had to post this from a friend's account.)
  50.  
  51. -- Chris
  52. cklarson@engr.ucdavis.edu
  53.