Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

A number of issues related to debugging #928

Closed
FullValueRider opened this issue Jun 21, 2022 · 1 comment
Closed

A number of issues related to debugging #928

FullValueRider opened this issue Jun 21, 2022 · 1 comment

Comments

@FullValueRider
Copy link

Describe the bug
Whist debugging I had occasion to try to view the contents of a dynamic array containing 16384 elements. I observed the following.

  1. It took more than a minute to show the elements
  2. The numbering of the array elements was string sequential and not numeric sequential

ArrayElementOrder

  1. If the view of the array elements was not closed then each subsequent F11/F10 in the method where the view was requested took a about a minute to complete.
  2. It was no possible to end the program using the run/end instruction (or I didn't wait tong enough) (I kept getting the 'main thread is busy' message
  3. The project could be saved and closed BUT when reopened, if the array view had not been previously closed , then the delays were present again when the code entered the method in which the array view had previously been requested.

Steps to reproduce the behaviour:

  1. Open the 'AoC2021NoLib' project
  2. start execution of main in MainModule using F6
  3. At each break point press F5 to continue
  4. The delay issues should manifest after the second F5
  5. The array being viewed was mKeys in Method Keys in Module cHashD

Expected behaviour

Array elements should be listed in numerical order of the index of the array elements not the string order

The delay to show the array contents of an array with 16k elements is unexpected, based on debugging in VBA and other languages I'd expect a delay of a couple of seconds at most.

The run/end code may need to be revised to avoid the problem of not being able to end a program.

Desktop (please complete the following information):

  • OS: Windows 11
  • twinBASIC compiler version Beta Ide 58

Additional context
These issues are not urgent, just a little frustrating.

twinBasic.zip

@WaynePhillipsEA
Copy link
Collaborator

There should be a slight improvement here in BETA 60 ;)

The remaining short delay now comes from the webpage rendering of the DOM list nodes, similar to the debug console performance from earlier issues. Ideally we need to group the array elements e.g. (0 - 100), (101 - 200) etc so that we can reduce the number of live DOM nodes and make accessing individual elements easier. I'll open another issue for that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants