I saw that guillermo already needed it for pillar Path >> pillarPrintString self isEmpty ifTrue: [ ^ '' ]. ^ String streamContents: [ :str | str nextPutAll: (self at: 1) asString. 2 to: self size do: [:i | str nextPutAll: self delimiter; nextPutAll: (self at: i) ] ] so there is clearly something wrong. On Fri, Feb 2, 2018 at 8:57 PM, Stephane Ducasse <stepharo.self@gmail.com> wrote:
Exactly, let us see what others think and we fix it.
To me FileSystem deserves some love.
On Fri, Feb 2, 2018 at 8:39 PM, Esteban A. Maringolo <emaringolo@gmail.com> wrote:
Hi Stef, all,
I've seen this "inconsistency" as well. The current printSting tries to be descriptive but isn't handy to work with.
I would go one step further and have no #asFileReference suffix nor File @ prefix, it is:
'/Users/ducasse/Workspace/FirstCircle/MyBooks/Bk-Writing/PharoBooks/Booklet-AMiniSchemeInPharo/_result' asFileReference printString
'/Users/ducasse/Workspace/FirstCircle/MyBooks/Bk-Writing/PharoBooks/Booklet-AMiniSchemeInPharo/_result'
If for some reason the option chosen is to keep the current printString then the printString should be:
'File @ ''/Users/ducasse/Workspace/FirstCircle/MyBooks/Bk-Writing/PharoBooks/Booklet-AMiniSchemeInPharo/_result'''
It is, a string with three tokens, the class name, the @ symbol and a string as an argument, that way you can evaluate the result back.
I prefer the first one, it enables you to work with files on workspaces, if you're using inspectors the printString doesn't matter much, since you're dealing with a specialized tool and there is no way you can confuse a FileReference with a regular String instance
Best regards, Esteban A. Maringolo
2018-02-02 16:12 GMT-03:00 Stephane Ducasse <stepharo.self@gmail.com>:
Hi
'/Users/ducasse/Workspace/FirstCircle/MyBooks/Bk-Writing/PharoBooks/Booklet-AMiniSchemeInPharo/_result' asFileReference printString
"File @ /Users/ducasse/Workspace/FirstCircle/MyBooks/Bk-Writing/PharoBooks/Booklet-AMiniSchemeInPharo/_result"
so either we add @ one File or we redefine printString one reference to be
'/Users/ducasse/Workspace/FirstCircle/MyBooks/Bk-Writing/PharoBooks/Booklet-AMiniSchemeInPharo/_result' asFileReference so that we can do something with the printString result.
What do you think? Guille is it what we talked together?
Setf