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

[Hernan Wilkinson]

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 WilkinsonSoftware Developer & Coach
>>>
>>> Alem 896, Floor 6, Buenos Aires, Argentina
>>>
>>> +54 11 6091 3125
>>>
>>> @HernanWilkinson
>>>
>>
>>
>> --
>> <https://10pines.com/>Hernán WilkinsonSoftware Developer & Coach
>>
>> Alem 896, Floor 6, Buenos Aires, Argentina
>>
>> +54 11 6091 3125
>>
>> @HernanWilkinson
>>
>
>
> --
>
> Nicolás Papagna
>


-- 
<https://10pines.com/>Hernán WilkinsonSoftware Developer & Coach

Alem 896, Floor 6, Buenos Aires, Argentina

+54 11 6091 3125

@HernanWilkinson
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.cuis.st/mailman/archives/cuis-dev/attachments/20220104/6764b9a8/attachment.htm>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 5033-DebuggerMethodMapFix-HernanWilkinson-2022Jan04-17h04m-HAW.001.cs.st
Type: application/octet-stream
Size: 1581 bytes
Desc: not available
URL: <http://lists.cuis.st/mailman/archives/cuis-dev/attachments/20220104/6764b9a8/attachment.obj>