FogBugz (Case [Issue]12319) Collection - Replace includesAllOf: by includesAll: everywhere.

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

FogBugz (Case [Issue]12319) Collection - Replace includesAllOf: by includesAll: everywhere.

Pharo Issue Tracker
FogBugz Notification
avatar
Clement Bera opened Case 12319: Replace includesAllOf: by includesAll: everywhere. and assigned it to Everyone:
Cleanup in Project:  Collection: 1. Pharo Image  •  You are subscribed to this case
So in Collection and its subclasses we have:

includesAllOf: aCollection

self flag: 'use includesAll: instead'.

^ self includesAll: aCollection

(I fixed a typo in the comment)

So clearly includesAllOf: is deprecated, recommending the use of includesAll: instead.

Now in TimeSpan, the same API is here, so I deprecated includesAllOf: recommending includesAll: (with a flad, not self deprecated, as in the collections)

Same issue for includesAny than for includesAll

Lastly I fixed the test name in TimespanTest:
testIncludesAllOf -> testIncludesAll
testIncludesAnyOf -> testIncludesAny
Priority Priority: 5 – Fix If Time Status Status: Work Needed
Assigned To Assigned to: Everyone Milestone Milestone: Pharo3.0

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

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

FogBugz

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