Manuscript (Case [Issue]22671) _Inbox - IdentitySet>>asSet returns an IdentitySet instance instead of Set

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

Manuscript (Case [Issue]22671) _Inbox - IdentitySet>>asSet returns an IdentitySet instance instead of Set

Pharo Issue Tracker
Manuscript Notification
avatar
Federico Mennite opened Case 22671: IdentitySet>>asSet returns an IdentitySet instance instead of Set and assigned it to Everyone:
Bug in Project:  _Inbox: Not Spam  •  You are subscribed to this case
Hi,
(IdentitySet with: Object new) asSet returns an instance of IdentitySet containing an object.
This behavior is surprising. I would expect to return an Instance of Set as the class it's only compatible but not the same.
There are multiple ways to fix that according to the semantics you want to honor.
A less more obvious issue with the current implementation is that this method isn't returning a copy of the original object, which of course is fast but arguably dangerous.
Priority Priority: 5 – Fix If Time Status Status: Work Needed
Assigned To Assigned to: Everyone Milestone Milestone: Pharo7.0

Go to Case
No longer need updates? Unsubscribe from this case.

Don't want Manuscript notifications anymore? Update your preferences.

Manuscript

_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
https://lists.gforge.inria.fr/mailman/listinfo/pharo-bugtracker