Feature request/suggestion
lazarus / 12-Debugger

----------------------------------------------------------------------
440bx · 2024-07-13 21:45

One of the rather nice Watches features is its ability to page arrays
and to set the index of the top of the page.

Unfortunately, it only accepts a decimal number as the top of the page
index.  It would be very nice if it also accepted the index in hex.

In the attachment, the xl_gaps, which are indexes in the xl_addresses
array are shown in hex.  The box that controls the top of page index
for xl_addresses does not accept a hex number making it a bit more
laborious to specify the index (got to mentally convert or change the
setting to show values in both hex and decimal which uses more space,
which is not always desirable.)

Thank you for reading and, comments welcome.

----------------------------------------------------------------------
Martin_fr · 2024-07-14 09:54

Noted. Currently they are SpinEdits (with hidden arrows). That way,
the numbers-only is already implemented.

Easy to change, just a matter of finding a bit of idle time...

ide\packages\idedebugger\arraynavigationframe.pas
    edArrayPageSize: TSpinEditEx;
    edArrayStart: TSpinEditEx;

They could also accept drag and drop...

They will *not* accept other watches though. Just literal numbers.

----------------------------------------------------------------------
440bx · 2024-07-14 11:52

Quote from: Martin_fr on July 14, 2024, 11:54:04 amThey will *not*
accept other watches though. Just literal numbers.
That would be just fine.

and... drag and drop would definitely be a very nice feature

ETA:

I probably know the answer to the question I'm going to ask but, what
the heck, should I create a feature request in the bug tracker ?

----------------------------------------------------------------------
Martin_fr · 2024-07-14 13:40

Quote from: 440bx on July 14, 2024, 01:52:42 pmI probably know the
answer to the question I'm going to ask but, what the heck, should I
create a feature request in the bug tracker ?

Yes, please.

And if you have time, you could even have a look at the file.
TSpinEdit is nothing special. Replacing it with a normal edit, and
making that numbers only is also not....

----------------------------------------------------------------------
440bx · 2024-07-14 14:25

Quote from: Martin_fr on July 14, 2024, 03:40:25 pmYes, please.
 Done.

Issue/request: 41027
Link: https://gitlab.com/freepascal.org/lazarus/lazarus/-/issues/41027

Quote from: Martin_fr on July 14, 2024, 03:40:25 pmAnd if you have
time, you could even have a look at the file. TSpinEdit is nothing
special. Replacing it with a normal edit, and making that numbers only
is also not....
when it comes to OOP stuff I am rather inadequate but, I'll have a
look.