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

Update for getCodeIndex() issue with GUI Builder for Processing (#545) #596

Closed
quarks opened this issue Nov 10, 2022 · 4 comments
Closed

Comments

@quarks
Copy link

quarks commented Nov 10, 2022

Issue 545 has been closed and locked so I have created this one.

I have released a new version of GUI Builder that no longer uses the method

getCodeIndex(SketchCode who)

so has resolved the original issue.

Sorry for any inconvenience 😄

@benfry
Copy link
Owner

benfry commented Nov 11, 2022

Hm, wonder if it's safe to remove again, or if anyone else is using it in a library? Any ideas?
b934c4d

@quarks
Copy link
Author

quarks commented Nov 11, 2022

GUI Builder creates a second tab for the GUI creation code and this tab is read and updated as the user switches between the sketch window and the GUI design window. To access the different tabs I needed to delve into the Sketch class source code to find the methods I needed and picked getCodeIndex(). All this was back in 2011 and it seems I picked a method that is not called by any other Processing code LOL.

GUI Builder now uses the methodgetCode() method, although undocumented I suspect Processing uses it otherwise it also would have been commented out 😄

I doubt that any other library or tool uses getCodeIndex() otherwise you would have heard either here or on the forum.

@benfry benfry closed this as completed in 1b137e2 Nov 11, 2022
@benfry
Copy link
Owner

benfry commented Nov 11, 2022

Got it, thanks for letting me know (and updating the Tool!)

@benfry benfry changed the title Update for issue 545 Update for getCodeIndex() issue with GUI Builder for Processing (issue #545) Nov 11, 2022
@benfry benfry changed the title Update for getCodeIndex() issue with GUI Builder for Processing (issue #545) Update for getCodeIndex() issue with GUI Builder for Processing (#545) Nov 11, 2022
@github-actions
Copy link

This issue has been automatically locked. To avoid confusion with reports that have already been resolved, closed issues are automatically locked 30 days after the last comment. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 12, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants