home *** CD-ROM | disk | FTP | other *** search
/ NetNews Usenet Archive 1993 #1 / NN_1993_1.iso / spool / comp / sys / mac / oop / macapp3 / 352 < prev    next >
Encoding:
Internet Message Format  |  1993-01-05  |  1.6 KB
  1. Path: sparky!uunet!olivea!apple!applelink.apple.com
  2. From: RSD@AppleLink.Apple.COM (Research SW Design, D Goldman,PRT)
  3. Newsgroups: comp.sys.mac.oop.macapp3
  4. Subject: Another one! (Part 1)
  5. Message-ID: <726228373.1142670@AppleLink.Apple.COM>
  6. Date: 5 Jan 93 09:59:00 GMT
  7. Sender: daemon@Apple.COM
  8. Organization: AppleLink Gateway
  9. Lines: 39
  10.  
  11. I'm not hunting for these, really! But here is another tiny bug that I have
  12. just stumbled upon, whose fix I again assert is completely safe to make BY
  13. INSPECTION, without any lengthy QA cycle.
  14.  
  15. I am posting these in part for the usual reason of alerting my fellow
  16. developers to problems they might encounter in their work. But in the context
  17. of this thread, I really do want to demonstrate that there are plenty of truly
  18. localized bugs that somebody could EASILY be fixing in a master copy of MacApp,
  19. and distributing to all of us on a periodic basis.
  20.  
  21. [Again, to avoid any hard feelings, I want to emphasize that the discussion of
  22. these really tiny bugs is not intended in any way as a criticism of MacApp or
  23. its creators and maintainers.]
  24.  
  25. The bug:
  26.  
  27.     In TTabber::Tab we find this:
  28.  
  29.          fNext->BecomeTarget();
  30.          fNext->SetTargetSelection(kRedraw);
  31.  
  32. The problem:
  33.  
  34.     If the current target refuses to resign, and therefore BecomeTarget()
  35.     returns FALSE, we will nevertheless scroll to bring the aborted new
  36.     target into sight. Depending on your situtation, this could actually
  37.     scroll the still-current target out of sight.
  38.  
  39. The solution:
  40.  
  41.          if ( fNext->BecomeTarget() )
  42.            fNext->SetTargetSelection(kRedraw);
  43.  
  44.  
  45. So, anybody have an argument with this one?
  46.  
  47. -- Dave Goldman
  48.    Research Software Design
  49.  
  50.