Re: [Screem-devel] Testing request
Status: Inactive
Brought to you by:
davek
From: David K. <da...@ri...> - 2000-02-19 19:30:44
|
Sven Liessem wrote: > Well I think I should have gone through the code before sending this > mail... As it stands "Select current context" searches for an > opening tag before the cursor position. Thus if the cursor is IN such > a tag the search goes ahead it and selects the enclosing context. > This is somehow disturbing for me and I would find it better to stop > the search immediately if we're already in a opening tag. I've changed the screem_editor_select_context() so that if the cursor is in an opening tag, then its that block that is selected, so: <hea|d> <title>this is a title</title> </head> Where the | is the cursor, would select all of the above as the context. The current_context function is also used in the auto closing tag feature, which is why I made the change in screem_editor_select_context() > BTW I had a strange behaviour when single stepping > screem_editor_select_context() with gdb : const gchar *mime_type and > gchar *text pointed to the same address after the call to > screem_editor_get_text() ! Don't really like this.... That is very strange behaviour, I can't even see how it could happen, as mime_type is set by calling screem_page_get_mime_type(), which returns a const char*, which is set when the page path is set from screem_page_set_pathname() by calling gnome_mime_type(), which itself returns a const char*. Whereas screem_page_get_text() returns a freshly malloced area of memory with the contents of the text widget, via gtk_editable_get_chars(). The problem I'm having with the select context is that it just seems unstable, and crashes on me after use, sometimes printing out a GDK-Error about a bad drawable. Although I think these are Gnome/GDK or even theme related problems rather than screem itself. Then again it could be related to whats happening with what you mention above. David -- Make your site SCREEM - Site Creating & Editing EnvironMent URL: http://www.screem.org/ Mail: da...@sc... |