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

FIX: Undo caused crash when inserting newline at the end of text #32

Open
wants to merge 27 commits into
base: master
Choose a base branch
from
Open

Conversation

Rasa0
Copy link

@Rasa0 Rasa0 commented Jun 14, 2018

Line 195 causes "vector subscript out of range" when undoing a newline on the last line.
Unsure if this is a proper fix or if it could be made better.

Rasa0 and others added 27 commits June 14, 2018 15:30
… reacting to clicks when pointing at the text.

Now auto scrolling up and down when dragging cursor outside the text area.
… the text as dirty. Useful for being able to save to file.
… highlight

Added the abillity to toggle breakpoints.
Added a breakpoints changed callback.
Fixed missing check for keywords starting with 'e'
Lexer now colors the code as it runns.
Now lexing numbers aswell
Now putting the name in name tokens
Now putting message in malformed number token
…e didn't initialize isVararg, causing all functions to be vararg
…ues.

Shows the values when variables are hovered now.
Now using a constant instead of magical value of -1 to indicate when program counter is disabled
Copy link
Owner

@BalazsJako BalazsJako left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is it really necessary to include imgui_internal.h? I would like to keep the source independent of ImGUI's internal structures.

@BalazsJako
Copy link
Owner

Whoa... that's a big list of changes. Is it possible to break it down into smaller ones? I would like to merge most of them, but not all, at least not at once. Can we discuss it please?

@Rasa0
Copy link
Author

Rasa0 commented Sep 5, 2018

Whops. I thought i made the fix based on the origin, must have been tired when i made the request. My current version have been heavily modified to fit my purposes (Focus on Lua only). And now it have some tight coupling to my private project making it unfit for sharing. I could try to split out the general parts into a public fork, but that will require some work reorganizing the commits.

The inclusion of imgui_internal.h was because I'm using ImRect.

@BalazsJako
Copy link
Owner

Ok, that would be great, I think it is enough to split it into a few separate commits, grouped by topic. Just let me know if you have done it.

I keep avoiding using inclusion of imgui_internal.h because I do not want to have any extra dependencies on ImGUI's internals, which might change every now and then and might cause extra work necessary every time the code is compiled against the new version of ImGUI.

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

Successfully merging this pull request may close these issues.

2 participants