Re: [Geany-Devel] Segmentation fault when auto-close plug-in is enable

2013-10-23 Thread Lex Trotman
Looking at the line that failed, first guess would be failure to check doc is valid before using it. Cheers Lex On 23 October 2013 20:20, n@sk0 arrted...@gmail.com wrote: There is strange behaviour when Auto-close plug-in is enabled. Steps to reproduce : 1. open Geany 2. Enable auto-close

Re: [Geany-Devel] Segmentation fault when auto-close plug-in is enable

2013-10-23 Thread Frank Lanitz
Am 23.10.2013 11:20, schrieb n@sk0: See attached gdb backtrace for more info. Can you create a backtrace for that event? (putting bt into gdb) Cheers, Frank signature.asc Description: OpenPGP digital signature ___ Devel mailing list

Re: [Geany-Devel] Segmentation fault when auto-close plug-in is enable

2013-10-23 Thread Frank Lanitz
Am 23.10.2013 15:03, schrieb Frank Lanitz: Am 23.10.2013 11:20, schrieb n@sk0: See attached gdb backtrace for more info. Can you create a backtrace for that event? (putting bt into gdb) Nevermind. Missed the real attachment. cheers, Frank signature.asc Description: OpenPGP digital

Re: [Geany-Devel] Segmentation fault when auto-close plug-in is enable

2013-10-23 Thread Frank Lanitz
Am 23.10.2013 11:20, schrieb n@sk0: There is strange behaviour when Auto-close plug-in is enabled. Which version of the plugin are you using here? signature.asc Description: OpenPGP digital signature ___ Devel mailing list Devel@lists.geany.org

Re: [Geany-Devel] Segmentation fault when auto-close plug-in is enable [patch]

2013-10-23 Thread Matthew Brush
On 13-10-23 05:29 AM, n@sk0 wrote: Before read : Keep in mind that i am *not* C/C++ native developer, and all message below can be just rant. After little testing and debugging, i found that : In on_editor_notify() function, user_data is not a valid pointer : on_editor_notify(GObject *obj, gint

Re: [Geany-Devel] Segmentation fault when auto-close plug-in is enable [patch]

2013-10-23 Thread Frank Lanitz
On Wed, 23 Oct 2013 07:53:16 -0700 Matthew Brush mbr...@codebrainz.ca wrote: The final problem is that, as Lex mentioned, it's not checking `DOC_VALID()` (or doc-is_valid) but just that data-doc != NULL, so if any document that was ever activated is closed, this is going to explode when the

Re: [Geany-Devel] Segmentation fault when auto-close plug-in is enable [patch]

2013-10-23 Thread Pavel Roschin
(for reasons I never understood, Geany recycles documents, so it's entirely possible to have a document pointer that is neither NULL nor valid) Didn't expect that. I'll fix my plugins for such kind of mistakes. -- Best regards, Pavel Roschin aka RPG

Re: [Geany-Devel] Segmentation fault when auto-close plug-in is enable [patch]

2013-10-23 Thread Thomas Martitz
Am 23.10.2013 16:53, schrieb Matthew Brush: On 13-10-23 05:29 AM, n@sk0 wrote: Before read : Keep in mind that i am *not* C/C++ native developer, and all message below can be just rant. After little testing and debugging, i found that : In on_editor_notify() function, user_data is not a valid

Re: [Geany-Devel] Segmentation fault when auto-close plug-in is enable

2013-10-23 Thread Thomas Martitz
Am 23.10.2013 11:20, schrieb n@sk0: There is strange behaviour when Auto-close plug-in is enabled. Steps to reproduce : 1. open Geany 2. Enable auto-close plug-in 3. open several files for edit 4. Click Ctrl+W (close document) -- Expected result : Currently opened document is closed. --

Re: [Geany-Devel] Segmentation fault when auto-close plug-in is enable

2013-10-23 Thread n@sk0
It works just fine now, thank you all. On 23/10/13 22:59, Thomas Martitz wrote: Am 23.10.2013 11:20, schrieb n@sk0: There is strange behaviour when Auto-close plug-in is enabled. Steps to reproduce : 1. open Geany 2. Enable auto-close plug-in 3. open several files for edit 4. Click Ctrl+W

Re: [Geany-Devel] Segmentation fault when auto-close plug-in is enable [patch]

2013-10-23 Thread Lex Trotman
On 24 October 2013 13:09, Matthew Brush mbr...@codebrainz.ca wrote: On 13-10-23 11:36 AM, Thomas Martitz wrote: [snip] Regarding that pattern we discussed previously and used in this AutoClose code for attaching data to a document, I'd be interested whether you or anyone thinks this

Re: [Geany-Devel] Segmentation fault when auto-close plug-in is enable [patch]

2013-10-23 Thread Lex Trotman
[...] I said it specifically to avoid getting into deep design discussions or code reviews, I just want a general +1 or -1 whether anyone thinks the general concept is useful and whether it's worth looking at implementing something similar. If not one else really cares, it's not worth getting

Re: [Geany-Devel] Segmentation fault when auto-close plug-in is enable [patch]

2013-10-23 Thread Matthew Brush
On 13-10-23 09:13 PM, Lex Trotman wrote: [...] I said it specifically to avoid getting into deep design discussions or code reviews, I just want a general +1 or -1 whether anyone thinks the general concept is useful and whether it's worth looking at implementing something similar. If not one

Re: [Geany-Devel] Segmentation fault when auto-close plug-in is enable [patch]

2013-10-23 Thread Matthew Brush
On 13-10-23 08:39 PM, Lex Trotman wrote: On 24 October 2013 13:09, Matthew Brush mbr...@codebrainz.ca wrote: On 13-10-23 11:36 AM, Thomas Martitz wrote: [snip] Regarding that pattern we discussed previously and used in this AutoClose code for attaching data to a document, I'd be

Re: [Geany-Devel] Segmentation fault when auto-close plug-in is enable [patch]

2013-10-23 Thread Lex Trotman
[...] But it seems a pity to have to duplicate the glib interface for each structure that offers the facility, can we just return the GData and let the plugins use the normal g_datalist functions? [...] To summarise the answer to the question Matthew gave on IRC, no, we don't have to