[Github-comments] [geany/geany] Crash after update to Scintilla 5.1.5 (Issue #3095)

[Jiří Techet]

> Weird I never saw a problem, just lucky I guess, although I did say not heavily tested.

I saw it because I tested the GDScript filetype with the example from here

https://docs.godotengine.org/en/stable/getting_started/scripting/gdscript/gdscript_basics.html

and playing with it (adding, removing and saving the result). I first thought it was something in the GDScript parser or lexer but then I reverted back to master and saw it too. Note that I'm running Geany in a arm64 virtual machine on a macBook and memory management can differ from the Intel one.

> It is likely best to change these 3 APIs to be consistent with others. However, this could break applications that will no longer allocate enough memory to include the NUL terminator. Downstream projects should check their use of these APIs.

@rdipardo Thanks for the pointer. This is why I'd first want to hear from Neil if he really considered it well or not. If it's meant to be this way, we'll have to deal with it.

> master is a development branch, nothing that has any guarantees for stability. 

Agree, the doomsday hasn't come yet :-).


-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/issues/3095#issuecomment-1009966122
You are receiving this because you are subscribed to this thread.

Message ID: <geany/geany/issues/3095/1009966122 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20220111/38254c26/attachment-0001.htm>