maziac / DeZog

Visual Studio Code Debugger for Z80/ZX Spectrum.
MIT License
210 stars 34 forks source link

Watch doesn't support byte group above 2 #56

Closed remy closed 3 years ago

remy commented 3 years ago

Describe the bug

If you try to watch a value grouped by 4 bytes (for 32bit) the result is wrong/corrupted.

To Reproduce

Steps to reproduce the behavior:

Add a watch for an address with 4 as the argument:

Screenshot 2021-04-25 at 10 07 46

Expected behavior

32bit groupings to be parsed. I would also expect 64bit - but I'm not sure what grouping by 3 bytes would do (or just a 24bit value?).

Version etc. (please complete the following information):

maziac commented 3 years ago

Could you please try this version v2.2.4: https://github.com/maziac/DeZog/releases/tag/v2.2.4

I have done some re-factoring. If you encounter any other problems please let me know.

remy commented 3 years ago

Looks good to me 👍