<div dir="ltr">I just checked the code and that behavior was introduced in the last version of that method, before it behaved as Nicola wanted, so thank you Nicola for pointing this out!<div><br></div><div>Juan, I think we should get that method back to its previous version. I did not look for when the change was made and I do not know who jpb is...</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, Aug 7, 2021 at 4:54 PM Juan Vuletich via Cuis-dev <<a href="mailto:cuis-dev@lists.cuis.st">cuis-dev@lists.cuis.st</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">I don't think the preference is needed. What about simply:<br>
<br>
selectionPrintString<br>
     "Returns the current selection as a string"<br>
     ^ self safelyPrintWith: [<br>
         self selection printTextLimitedTo: self printStringLimit ].<br>
<br>
?<br>
<br>
Thanks,<br>
<br>
-- <br>
Juan Vuletich<br>
<a href="http://www.cuis-smalltalk.org" rel="noreferrer" target="_blank">www.cuis-smalltalk.org</a><br>
<a href="https://github.com/Cuis-Smalltalk/Cuis-Smalltalk-Dev" rel="noreferrer" target="_blank">https://github.com/Cuis-Smalltalk/Cuis-Smalltalk-Dev</a><br>
<a href="https://github.com/jvuletich" rel="noreferrer" target="_blank">https://github.com/jvuletich</a><br>
<a href="https://www.linkedin.com/in/juan-vuletich-75611b3" rel="noreferrer" target="_blank">https://www.linkedin.com/in/juan-vuletich-75611b3</a><br>
@JuanVuletich<br>
<br>
<br>
On 8/6/2021 2:17 PM, Gerald Klix via Cuis-dev wrote:<br>
> Nicola,<br>
><br>
> I always thought this is on purpose, but did<br>
> not like it.<br>
> If you look at Inspector>>#selectionPrintString<br>
><br>
> --- snip ---<br>
> selectionPrintString<br>
>     "Returns the current selection as a string"<br>
>     ^self safelyPrintWith: [<br>
>         | selectedValue |<br>
>         selectedValue _ self selection.<br>
>         (selectedValue is: #String)<br>
>             ifTrue: [ selectedValue ]<br>
>             ifFalse: [ selectedValue printTextLimitedTo: self <br>
> printStringLimit]]<br>
> --- snap ---<br>
> it has a special case for strings, confirming my suspicion.<br>
><br>
> If you use this method implementation instead<br>
> --- Schnipp ---<br>
> selectionPrintString<br>
>     "Returns the current selection as a string"<br>
>     ^self safelyPrintWith: [<br>
>         | selectedValue |<br>
>         selectedValue _ self selection.<br>
>         (selectedValue is: #String) and: [ Preferences <br>
> inspectStringsLiteraly ] ::<br>
>             ifTrue: [ selectedValue ]<br>
>             ifFalse: [ selectedValue printTextLimitedTo: self <br>
> printStringLimit]]<br>
> --- Schnapp ---<br>
> we will get the behavior you desired.<br>
> Please note the inspectStringsLiteraly-method, that<br>
> defaults to false when not implemented, by virtur<br>
> of Preferences DNU logic.<br>
><br>
> The attached change set makes the inspector configurable<br>
> but retains the old behavior.<br>
> Maybe we can convince Juan to integrate that improvement.<br>
><br>
> HTH and Best Regards,<br>
><br>
> Gerald<br>
><br>
><br>
> On 8/6/21 6:14 PM, Nicola Mingotti via Cuis-dev wrote:<br>
>><br>
>> Hi guys,<br>
>><br>
>> I hit a stupid bug today ... to find it was nasty, i think because<br>
>> the debugger does not show what (at least I) would expect.<br>
>><br>
>> Please see the attached picture here:<br>
>> <a href="https://pasteboard.co/KeEwgAy.png" rel="noreferrer" target="_blank">https://pasteboard.co/KeEwgAy.png</a><br>
>><br>
>> If you look at the debugger bottom right pane you think<br>
>> that 'aStringRepr' is nil, right? Well, it is not so,<br>
>> If you inspect it, see inspector in the middle of the screen,<br>
>> you see it actually is 'nil', that is a String containing a 3 letter <br>
>> value.<br>
>><br>
>> This happens also for numbers, if they are inside Strings the debugger<br>
>> does not show eg: '123', but 123. That is some kind of "print <br>
>> representation".<br>
>><br>
>> I think this is very much undesirable and an unexpected behavior,<br>
>> at least for a beginner;)<br>
>><br>
>> bye<br>
>> Nicola<br>
>><br>
-- <br>
Cuis-dev mailing list<br>
<a href="mailto:Cuis-dev@lists.cuis.st" target="_blank">Cuis-dev@lists.cuis.st</a><br>
<a href="https://lists.cuis.st/mailman/listinfo/cuis-dev" rel="noreferrer" target="_blank">https://lists.cuis.st/mailman/listinfo/cuis-dev</a><br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><span style="font-size:xx-small;border-collapse:collapse"><div style="font-size:small"><a href="https://10pines.com/" style="font-family:Roboto,Helvetica,Arial,sans-serif;font-size:medium" target="_blank"><img width="108" src="https://10pines.github.io/email-signature/10pines-firma@2x.png" style="margin-bottom: 0.5em;"></a><span style="color:rgb(0,0,0);font-family:Roboto,Helvetica,Arial,sans-serif;font-size:medium"></span><h1 style="margin:0px;font-size:14px">Hernán Wilkinson</h1><h2 style="margin:0px 0px 1em;font-size:14px;color:rgb(100,100,100)">Software Developer, Teacher & Coach</h2><p style="margin:0px;color:rgb(100,100,100);font-size:12px">Alem 896, Floor 6, Buenos Aires, Argentina</p><p style="margin:0px;color:rgb(100,100,100);font-size:12px">+54 11 6091 3125</p><p style="margin:0px;color:rgb(100,100,100);font-size:12px">@HernanWilkinson</p></div></span></div></div>