FogBugz (Case [Issue]12205) Crosscutting and Misc - Cleanup gofer>>#directory:

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

FogBugz (Case [Issue]12205) Crosscutting and Misc - Cleanup gofer>>#directory:

Pharo Issue Tracker
FogBugz Notification
avatar
Sean DeNigris opened Case 12205: Cleanup gofer>>#directory: and assigned it to Sean DeNigris:
Cleanup in Project:  Crosscutting and Misc: 1. Pharo Image  •  You are subscribed to this case
Discussion started at http://forum.world.st/Pharo-s-Gofer-td4723799.html :

<code>Gofer>>#directory:
...
Smalltalk globals
at: #MCSubDirectoryRepository
ifAbsent: [ self error: aDirectoryOrString printString , ' is an unsupported repository type' ]</code>

Why wouldn't MCSubDirectoryRepository be present? Are we staying compatible with other platforms? Or another reason? If it's unnecessary I will gut... err, I mean simplify, hehe...

AND...

<code>Gofer>>#directory:
...
repository := (aDirectoryOrString isString and: [ aDirectoryOrString endsWith: '*' ])</code>

I don't like the double-duty of appending $* to the string argument to indicate a subdirectory repository. How about adding to the API, like directoriesIn: or similar...
Priority Priority: 5 – Fix If Time Status Status: Work Needed
Assigned To Assigned to: Sean DeNigris 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