[Cuis-dev] [BUG] "DoIt' does not work in the Debugger

[Nicolás Papagna Maldonado]

Yay! Thanks for fixing it @Hernan Wilkinson <hernan.wilkinson at 10pines.com>!

Cheers,
Nico PM

On Wed, Jan 5, 2022 at 11:56 AM Juan Vuletich <JuanVuletich at zoho.com> wrote:

> Amazing!
>
> Thanks Hernán. Works great. Now at GitHub.
>
> Cheers,
>
> On 1/4/2022 5:08 PM, Hernan Wilkinson via Cuis-dev wrote:
>
> Well... believe it or not I could fix it... the truth is that it was
> working when reading variables but not when assigning them, so I reused the
> reading var code from the assignment one :-)
>
> The .cs fixes the problem!!
>
> Cheers
> Hernan.
>
>
> On Mon, Jan 3, 2022 at 5:32 PM Nicolás Papagna Maldonado <
> nicolas.papagna at gmail.com> wrote:
>
>> Thanks for taking a look at it!
>>
>> Yes, that was my hunch. I didn't think it was so hard to fix tho.
>>
>> If that is the case, I believe we should "fail gratefully" to the user
>> instead of showing a low-level error like that (parser, etc).
>> That will keep us from giving a bad UX/feeling that "the system is
>> broken".
>>
>> What do you think?
>>
>> Best,
>> Nico PM
>>
>> On Mon, Jan 3, 2022 at 5:15 PM Hernan Wilkinson <
>> hernan.wilkinson at 10pines.com> wrote:
>>
>>> This error exists since variables are automatically defined in the
>>> Workspace.
>>> The problem is that those variables are not defined as temporary
>>> variables but as global variables in the workspace, although the DoIt
>>> bytecode treats them as temporary variables and that is the reason it does
>>> not find them.
>>>
>>> The solution would be for the workspace to define the variable as local,
>>> not global, but to do that major changes in the parser and workspace should
>>> be done and variables in the workspace could not be inspected after
>>> execution because they would be local and not global... It is a very
>>> difficult and time consuming change for a rare situation like this one.
>>> Another solution would be for the parser to realize they are not local
>>> vars but global vars and create the corresponding bytecode... another
>>> really time consuming task, not easy at all...
>>>
>>> I think that it does not make sense to invest time on this error by now
>>> (I've already invested 2.5 hours trying different solutions...)
>>>
>>> Cheers!
>>> Hernan.
>>>
>>>
>>> On Mon, Jan 3, 2022 at 11:59 AM Hernan Wilkinson <
>>> hernan.wilkinson at 10pines.com> wrote:
>>>
>>>> ok, the error happens when the variable (in the example a) is not
>>>> defined in the workspace. If it is defined, it works...
>>>> Let me look deeper.
>>>>
>>>> Cheers!
>>>> Hernan.
>>>>
>>>> PS: Nice way to start the year! :-)
>>>>
>>>> On Sun, Jan 2, 2022 at 1:35 PM Nicolás Papagna Maldonado via Cuis-dev <
>>>> cuis-dev at lists.cuis.st> wrote:
>>>>
>>>>> Aloha folks, and happy 2022 to everyone!
>>>>>
>>>>> I filed this bug yesterday and forgot to send it to the list: Invoking
>>>>> Do it in a Debugger raises an error
>>>>> <https://github.com/Cuis-Smalltalk/Cuis-Smalltalk-Dev/issues/220>
>>>>>
>>>>> Best,
>>>>> Nico PM
>>>>> --
>>>>> Cuis-dev mailing list
>>>>> Cuis-dev at lists.cuis.st
>>>>> https://lists.cuis.st/mailman/listinfo/cuis-dev
>>>>>
>>>>
>>>>
>>>> --
>>>> <https://10pines.com/> Hernán Wilkinson Software Developer & Coach
>>>>
>>>> Alem 896, Floor 6, Buenos Aires, Argentina
>>>>
>>>> +54 11 6091 3125
>>>>
>>>> @HernanWilkinson
>>>>
>>>
>>>
>>> --
>>> <https://10pines.com/> Hernán Wilkinson Software Developer & Coach
>>>
>>> Alem 896, Floor 6, Buenos Aires, Argentina
>>>
>>> +54 11 6091 3125
>>>
>>> @HernanWilkinson
>>>
>>
>>
>> --
>>
>> Nicolás Papagna
>>
>
>
> --
> <https://10pines.com/> Hernán Wilkinson Software Developer & Coach
>
> Alem 896, Floor 6, Buenos Aires, Argentina
>
> +54 11 6091 3125
>
> @HernanWilkinson
>
>
>
> --
> Juan Vuletichwww.cuis-smalltalk.orghttps://github.com/Cuis-Smalltalk/Cuis-Smalltalk-Devhttps://github.com/jvuletichhttps://www.researchgate.net/profile/Juan-Vuletichhttps://www.linkedin.com/in/juan-vuletich-75611b3https://independent.academia.edu/JuanVuletich
> @JuanVuletich
>
>

-- 

Nicolás Papagna
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.cuis.st/mailman/archives/cuis-dev/attachments/20220105/1f651da2/attachment-0001.htm>