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

Replace expression with DOC_VALID #7

Closed
wants to merge 2 commits into from
Closed

Replace expression with DOC_VALID #7

wants to merge 2 commits into from

Conversation

andy5995
Copy link
Owner

@elextr I still need the extra 'doc->file_name' check otherwise Geany crashes if the user has an 'untitled' document in front of them. Not a typical use case, but I encountered the crash by accident just a couple days after using the plug-in.

(Related to discussion at geany/geany-plugins#1308 (comment))

@elextr I still need the extra 'doc->file_name' check otherwise Geany
crashes if the user has an 'untitled' document in front of them. Not a
typical use case, but I encountered the crash by accident just a couple
days after using the plug-in.
@andy5995
Copy link
Owner Author

Completed in 9312764

@andy5995 andy5995 closed this Feb 20, 2024
@andy5995 andy5995 deleted the try-doc-valid branch February 20, 2024 22:22
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.

None yet

2 participants