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.s...@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
> <emaring...@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.s...@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
>>>
>>

Reply via email to