|Anonymous | Login||06-26-2019 10:33 UTC|
|Main | My View | View Issues | Change Log | Docs|
|Viewing Issue Simple Details [ Jump to Notes ]||[ View Advanced ] [ Issue History ] [ Print ]|
|ID||Category||Severity||Reproducibility||Date Submitted||Last Update|
|0007101||[Squeak] Janitorial||minor||always||06-24-08 20:53||04-18-10 22:04|
|Reporter||nicolas cellier||View Status||public|
|Assigned To||nicolas cellier|
|Summary||0007101: asSmallPositiveDegrees uselessly complex and can loop infinitely|
Try for example:
Current code is:
| result |
result _ self.
[result < 0] whileTrue: [result _ result + 360].
^ result \\ 360
when it could be:
^ self \\ 360
Anyway, this message does not seem often sent...
(0012313 - 724 - 758 - 758 - 758 - 758 - 758)
Yes, obviously this is a careless (and even comical) oversight, which never was exposed because this is a method which in practice is never called and which occupies a disused branch of effort dating from ancient SqC days, which really has no role in any squeak.org image. (Labelling this item 'Janitorial' is very appropriate...)
Note that the nearby, and much earlier, #asSmallAngleDegrees, which -- unlike #asSmallPositiveDegrees -- actually is still actively used in the system, is by the same author, and illustrates that he actually sometimes knows better ;-)
In any case, as you say, the method should surely be changed to have its entire implementation be "^ self \\ 360" in any image in which it is retained.
(0012317 - 132 - 138 - 138 - 138 - 138 - 138)
'Janitorial' labelling is a eye-hand-mouse loop failure, 'Kernel' was wanted.
Should think of a revision of my neuro control system
(0013353 - 59 - 59 - 209 - 209 - 209 - 209)
|Fixed in http://source.squeak.org/trunk/Kernel-nice.259.mcz [^]|
|06-24-08 20:53||nicolas cellier||New Issue|
|06-29-08 00:48||ScottWallace||Note Added: 0012313|
|07-02-08 22:55||nicolas cellier||Note Added: 0012317|
|10-05-09 19:37||nicolas cellier||Issue Monitored: nicolas cellier|
|10-05-09 19:37||nicolas cellier||Status||new => resolved|
|10-05-09 19:37||nicolas cellier||Fixed in Version||=> trunk|
|10-05-09 19:37||nicolas cellier||Resolution||open => fixed|
|10-05-09 19:37||nicolas cellier||Assigned To||=> nicolas cellier|
|10-05-09 19:37||nicolas cellier||Note Added: 0013353|
|04-18-10 22:04||andreas||Status||resolved => closed|
| Mantis 1.0.8[^]
Copyright © 2000 - 2007 Mantis Group
48 total queries executed.|
36 unique queries executed.