Re: [Geany-devel] Remove MSYS dependency of Geany on Win~1

2012-07-26 Thread Nick Treleaven
On 08/07/2012 13:04, Nick Treleaven wrote: BTW I've run into 2 problems with the install target: 1. cp -r and xcopy treat the destination path differently. Currently MSYS install is broken and will install data files as $DESTDIR/data/data/* - I've replaced xcopy with separate copy targets

Re: [Geany-devel] Dropping Waf support?

2012-07-18 Thread Nick Treleaven
On 16/07/2012 18:36, Enrico Tröger wrote: the make/MSYS build system support seems to get better and better due to Nick's and Dimitar's work on it Well, it's quite basic and needs a bit more work like allowing overridden CFLAGS. It doesn't install files like README, geany.html, etc. It also

Re: [Geany-devel] Remove MSYS dependency of Geany on Win~1

2012-07-10 Thread Nick Treleaven
On 09/07/2012 19:13, Dimitar Zhekov wrote: On Sun, 08 Jul 2012 13:04:54 +0100 Nick Treleavennick.trelea...@btinternet.com wrote: BTW I've run into 2 problems with the install target: 1. cp -r and xcopy treat the destination path differently. Currently MSYS install is broken and will install

Re: [Geany-devel] Remove MSYS dependency of Geany on Win~1

2012-07-08 Thread Nick Treleaven
On 07/07/2012 13:16, Dimitar Zhekov wrote: On Wed, 04 Jul 2012 12:59:11 +0100 Nick Treleavennick.trelea...@btinternet.com wrote: On 28/06/2012 18:55, Dimitar Zhekov wrote: BTW if you think plugins$(DIRSEP)*.dll looks ugly, we can use: plugins$/*.dll Nice. :) Though what I find a bit

Re: [Geany-devel] Any suggestions where message is from?

2012-07-08 Thread Nick Treleaven
On 07/07/2012 19:01, Colomban Wendling wrote: Compile with `-DG_FATAL_WARNINGS` and then run in gdb. It'll abort and you can see where it happened. …or run in GDB with G_DEBUG=fatal-warnings envvar set. Personally I use gdb with: r --g-fatal-warnings bt Nick

Re: [Geany-devel] Remove MSYS dependency of Geany on Win~1

2012-07-04 Thread Nick Treleaven
On 28/06/2012 18:55, Dimitar Zhekov wrote: On Thu, 28 Jun 2012 14:41:25 +0100 Nick Treleavennick.trelea...@btinternet.com wrote: Yes, using a target for plugins only would be OK. What if we put the an install: target in plugins/makefile.win32? That works, needs no DIRSEP, and is very

Re: [Geany-devel] Remove MSYS dependency of Geany on Win~1

2012-06-28 Thread Nick Treleaven
On 27/06/2012 21:32, Dimitar Zhekov wrote: On Tue, 26 Jun 2012 21:21:57 +0100 Nick Treleavennick.trelea...@btinternet.com wrote: I've now committed to master an install target similar to your first patch, but using xcopy for data. I used a DIRSEP variable: $(CP) plugins$(DIRSEP)*.dll

Re: [Geany-devel] Remove MSYS dependency of Geany on Win~1

2012-06-27 Thread Nick Treleaven
On 26/06/2012 21:46, Nick Treleaven wrote: On 26/06/2012 21:31, Nick Treleaven wrote: On 26/06/2012 21:21, Nick Treleaven wrote: MSYS does treat backslashes as escapes, and I think that is a worse problem than cmd.exe interpreting forward slashes as command options, as it can occur in more

Re: [Geany-devel] Remove MSYS dependency of Geany on Win~1

2012-06-26 Thread Nick Treleaven
On 25/06/2012 20:33, Dimitar Zhekov wrote: Hi. Here is a small diff (for makefile.win32 and src/makefile.win32 only) that makes Geany 1.22 Win~1 compilation and installation independent of MSYS. Usage: C mingw32-make -f makefile.win32 C mingw32-make -f makefile.win32 install Tested with

Re: [Geany-devel] Remove MSYS dependency of Geany on Win~1

2012-06-26 Thread Nick Treleaven
On 26/06/2012 20:06, Dimitar Zhekov wrote: MSYS does treat backslashes as escapes, and I think that is a worse problem than cmd.exe interpreting forward slashes as command options, as it can occur in more situations. I think we should use forward slashes throughout. I expected something like

Re: [Geany-devel] Remove MSYS dependency of Geany on Win~1

2012-06-22 Thread Nick Treleaven
On 19/06/2012 22:25, Matthew Brush wrote: On 12-06-19 10:12 AM, Dimitar Zhekov wrote: Hi, Now that 1.22 is out, how about removing the MSYS build dependency under Win~1? I tried to compile Geany with the default MinGW make, without any MSYS, and there were some easily fixable problems: cd foo

Re: [Geany-devel] [geany/geany] Question on - 7b3b65: Add workaround for users with an invisible selection style

2012-06-08 Thread Nick Treleaven
On 07/06/2012 00:11, Lex Trotman wrote: Those people should now fix their color schemes. I could lower the severity to an info message. Don't think it makes much difference, up to you. But we should add it Info messages are not shown on the console with recent GLib AIUI. to the

Re: [Geany-devel] [geany/geany] Question on - 7b3b65: Add workaround for users with an invisible selection style

2012-06-06 Thread Nick Treleaven
On 05/06/2012 06:10, Lex Trotman wrote: On 5 June 2012 01:30, Nick Treleavennick.trelea...@btinternet.com wrote: On 04/06/2012 07:59, Lex Trotman wrote: Based on your answer on my ML question is this necessary? Now that we always apply the settings, a selection style of false,false will

Re: [Geany-devel] [geany/geany] Question on - 7b3b65: Add workaround for users with an invisible selection style

2012-06-04 Thread Nick Treleaven
On 04/06/2012 07:59, Lex Trotman wrote: Based on your answer on my ML question is this necessary? Now that we always apply the settings, a selection style of false,false will reset to the Scintilla default The default is to show the selection by changing the background to light gray and leaving

Re: [Geany-devel] Selection and colour schemes

2012-06-03 Thread Nick Treleaven
On 02/06/2012 07:27, Lex Trotman wrote: Hi All, I just pushed a fix for switching colour schemes from say dark to default leaving selection unchanged. But in reality the code at highlighting.c doesn't actually do what it advertises, the comment says to override default which matches the

Re: [Geany-devel] tagmanager changes

2012-05-07 Thread Nick Treleaven
On 02/05/2012 05:46, Lex Trotman wrote: Hi All, To summarise since the thread has several subthreads. 1. Tagmanager Understandability a. I generated the doxygen documentation for tagmanager, it works if you set recursive, but didn't help much: - if its not OOP why does it say things like

Re: [Geany-devel] tagmanager changes

2012-05-07 Thread Nick Treleaven
On 07/05/2012 17:04, Nick Treleaven wrote: On 02/05/2012 05:46, Lex Trotman wrote: - its not clear how it all goes together, the workspace contains global tags and work_objects, or is that files and whats the workspace work objects are document tags. global tags explained in geany's manual

Re: [Geany-devel] Request: multithreaded tag generation?

2012-05-07 Thread Nick Treleaven
On 07/11/2011 15:35, Harold Aling wrote: I recently switched from GeanyPRJ to Gproject. Since Gproject doesn't support multiple open projects I have to switch between projects, but it takes up to 4 minutes to close one project and open another. A project consists of roughly 1000-2000 php-related

Re: [Geany-devel] tagmanager changes

2012-04-30 Thread Nick Treleaven
On 29/04/2012 14:07, Lex Trotman wrote: On 29 April 2012 22:07, Nick Treleavennick.trelea...@btinternet.com wrote: On 27/04/2012 06:30, Lex Trotman wrote: I don't see this myself, I see some complicating issues which could be resolved (and I'm willing to work on them), but generally a sound

Re: [Geany-devel] tagmanager changes

2012-04-30 Thread Nick Treleaven
On 29/04/2012 15:42, Colomban Wendling wrote: * it support asynchronous parsing (though not concurrent parsing); What's the difference? Also, what does it buy us? What I meant when saying it's asynchronous but not concurrent is that it supports launching the parsers in a separate thread, but

Re: [Geany-devel] tagmanager changes

2012-04-30 Thread Nick Treleaven
On 29/04/2012 15:47, Colomban Wendling wrote: Le 26/04/2012 18:53, Nick Treleaven a écrit : On 26/04/2012 16:02, Nick Treleaven wrote: On 24/04/2012 22:31, Colomban Wendling wrote: * it uses the same tag parsers tagmanager used, in ctagsmanager/ctags; BTW this is a good idea to clearly

Re: [Geany-devel] tagmanager changes

2012-04-29 Thread Nick Treleaven
On 27/04/2012 06:30, Lex Trotman wrote: [...] I don't understand why tagmanager has to be replaced, why not just replace the parts you want to improve? Rewriting it is likely to lead to a new set of bugs and be hard to review and merge changes from master. One of the problems with

Re: [Geany-devel] tagmanager changes

2012-04-26 Thread Nick Treleaven
On 24/04/2012 22:31, Colomban Wendling wrote: Le 17/04/2012 18:20, Nick Treleaven a écrit : Sorry for the long delays -- and also small activity -- recently. I have/had a lot of non-Geany stuff to do and stuff, the whole story, you know. No problem. I finally committed it and pushed it so

Re: [Geany-devel] tagmanager changes

2012-04-26 Thread Nick Treleaven
On 26/04/2012 16:02, Nick Treleaven wrote: On 24/04/2012 22:31, Colomban Wendling wrote: * it uses the same tag parsers tagmanager used, in ctagsmanager/ctags; BTW this is a good idea to clearly separate CTags from tagmanager. If this change can be applied separately, perhaps it could

Re: [Geany-devel] Standard Tags files

2012-04-25 Thread Nick Treleaven
On 25/04/2012 03:35, Lex Trotman wrote: Hi All, Nick has removed the instructions on re-building tags.c99 which is ok. But it made me think, should we provide a tags.c11 as well and how would it be best to choose between them? IIUC C11 adds a significant number of libraries, so extra tags

Re: [Geany-devel] Small additions to stash

2012-04-06 Thread Nick Treleaven
On 05/04/2012 19:02, Dimitar Zhekov wrote: New stash_group_free_strings(), which frees any strings and string arrays in a group. Much easier than to track them individually, as in Geany (though if someone is willing to track the keyfile_groups prefs free-s and remove them, a call may

Re: [Geany-devel] Small additions to stash

2012-04-05 Thread Nick Treleaven
On 04/04/2012 19:58, Dimitar Zhekov wrote: Hi, How about some $subject? Namely: Include stash_group_set_use_defaults() in plugin StashFuncs. I'm not using it any more, but it's a nice thing to have. I'm not convinced we should support that publicly. It puts a burden on the implementation

Re: [Geany-devel] Gathering Release 1.22 incompatibilities

2012-03-20 Thread Nick Treleaven
On 20/03/2012 04:46, Lex Trotman wrote: +Filetypes +* Move all named styles into color schemes and keep only mappings in the + filetypes files. Filetypes should no longer contain styling information, + except `filetypes.common` which contains the default color scheme. Breaks +

Re: [Geany-devel] Gathering Release 1.22 incompatibilities

2012-03-19 Thread Nick Treleaven
On 19/03/2012 01:41, Lex Trotman wrote: On 19 March 2012 12:12, Matthew Brushmbr...@codebrainz.ca wrote: On 12-03-18 05:20 PM, Lex Trotman wrote: Hi All, I recently ran into Nicks change of the default keybindingctrl-t from transpose to gototag. This made me realise we need to keep a list

Re: [Geany-devel] [geany/geany] 7cc443: Don't append file truncation warning if file doesn't exist - incorrect for remote files

2012-02-06 Thread Nick Treleaven
On 01/02/2012 23:05, Lex Trotman wrote: On Thu, Feb 2, 2012 at 3:51 AM, Nick Treleaven nick.trelea...@btinternet.com wrote: On 31/01/2012 23:09, Lex Trotman wrote: Confusing the user is also harm. A (paranoid) user may worry some other file got truncated. Ok, better put the filename

Re: [Geany-devel] [geany/geany] 7cc443: Don't append file truncation warning if file doesn't exist - incorrect for remote files

2012-01-31 Thread Nick Treleaven
On 27/01/2012 21:36, Lex Trotman wrote: Anyway, I don't know how to improve the situation. My commit is necessary to avoid confusing users about truncation on a local file that doesn't even No, neither do I, but we do get complaints about truncated files on network shares/gvfs every so often,

Re: [Geany-devel] [geany/geany] 7cc443: Don't append file truncation warning if file doesn't exist - incorrect for remote files

2012-01-27 Thread Nick Treleaven
On 26/01/2012 22:32, Lex Trotman wrote: If writing a remote file failed half way because of a network failure, then if the network is still faulty when we g_file_test for a regular file, it will fail because the file doesn't exist, but the partly written file may (is likely to) be truncated.

Re: [Geany-devel] encoding combo boxes bug - utility functions

2012-01-27 Thread Nick Treleaven
On 26/01/2012 05:21, Lex Trotman wrote: On Thu, Jan 26, 2012 at 3:41 PM, Matthew Brushmbr...@codebrainz.ca wrote: On 01/25/2012 06:45 PM, Lex Trotman wrote: [...] Hi Matthew, [...] utils_is_uri() - good utility function, well named Indeed well named and probably a little clearer that

Re: [Geany-devel] encoding combo boxes bug - utility functions

2012-01-25 Thread Nick Treleaven
On 24/01/2012 03:30, Matthew Brush wrote: I probably don't know 40%+ of Geany's code after casually hacking on it for well over a year. When reading the code, I have to refer to the source file for each function called to see what it does, with GTK+ I've already done this for many cases, and

Re: [Geany-devel] glade diffs - Re: encoding combo boxes bug

2012-01-25 Thread Nick Treleaven
On 22/01/2012 13:01, Nick Treleaven wrote: On 20/01/2012 18:39, Colomban Wendling wrote: @Nick: how did you generate the Glade file for 21cd7bb2139fd67644e5777bb8e6387d34473d09 Add Project overrides for 'Saving files' checkbox options? When I do use Glade 3.8.1 do modify the file it keeps

Re: [Geany-devel] encoding combo boxes bug

2012-01-22 Thread Nick Treleaven
On 20/01/2012 21:29, Matthew Brush wrote: @All: I added ui_builder_get_object() to be able to fetch a non-widget (list store here) from prefs.c, but I'm not completely sure it's the prefect fix. If you have any idea on how to improve this, spread them! :) IMO, it'd be better to just move the

[Geany-devel] glade diffs - Re: encoding combo boxes bug

2012-01-22 Thread Nick Treleaven
On 20/01/2012 18:39, Colomban Wendling wrote: @Nick: how did you generate the Glade file for 21cd7bb2139fd67644e5777bb8e6387d34473d09 Add Project overrides for 'Saving files' checkbox options? When I do use Glade 3.8.1 do modify the file it keeps reordering prefs/project dialogs... Just

Re: [Geany-devel] SCSS (Sassy CSS) lexer support

2012-01-18 Thread Nick Treleaven
On 17/01/2012 20:36, Lex Trotman wrote: I think if all is needed is to copy SciLexer.h and the new lexer file, plus minor makefile changes, then a patch should be acceptable without upgrading Scintilla. We've done this in the past. Hi Nick, How does this work without a filetype to support

Re: [Geany-devel] Adding some Glade widgets

2012-01-09 Thread Nick Treleaven
On 09/01/2012 00:56, Matthew Brush wrote: - GeanyWrapLabel - Nice little existing GtkWidget that fixes broken GtkLabel text wrapping in GTK+ 2. Should work fine in a glade catalog and should even fix wrapping of labels in Glade. This would be really nice to have, I hate having to create

Re: [Geany-devel] Weird Segfault Crash

2012-01-02 Thread Nick Treleaven
On 02/01/2012 14:33, Colomban Wendling wrote: encodings_convert_to_utf8_from_charset(utf8_name, (gsize) -1, ...) Is it OK the cast a negative number to `gsize` and will it have the desired effect for that function? The `-1` here is supposed to tell the encoding function that the

[Geany-devel] Glade 3 version?

2011-12-19 Thread Nick Treleaven
Hi, I tried opening data/geany.glade with the latest Glade, 3.8.1 on Windows. Pressing Save writes a lot of changes to the file, 260 Kb. It seems to be mostly reordering property tag lines for use_action_appearance. Also, when I added 2 menu items it created duplicate image ids for image1,

Re: [Geany-devel] Glade 3 version?

2011-12-19 Thread Nick Treleaven
On 19/12/2011 14:40, Matthew Brush wrote: On 12/19/2011 05:54 AM, Nick Treleaven wrote: Hi, I tried opening data/geany.glade with the latest Glade, 3.8.1 on Windows. Pressing Save writes a lot of changes to the file, 260 Kb. It seems to be mostly reordering property tag lines

Re: [Geany-devel] Stub project files for sharing

2011-11-11 Thread Nick Treleaven
On 08/11/2011 21:11, Lex Trotman wrote: [...] Unfortunately this is a requirement for any overriding scheme like stubs, if all the settings are written to the users project file no settings will be read from the stub file. No, what I would do is: * Read the user config file content * Append

Re: [Geany-devel] Short function factoring

2011-11-10 Thread Nick Treleaven
On 08/11/2011 21:21, Lex Trotman wrote: On Wed, Nov 9, 2011 at 3:40 AM, Nick Treleaven nick.trelea...@btinternet.com wrote: On 04/11/2011 00:21, Lex Trotman wrote: * long functions cause bugs Rubbish, wrong functions cause bugs no matter how long, and being unable to see the whole logic

Re: [Geany-devel] Short function factoring

2011-11-10 Thread Nick Treleaven
On 09/11/2011 02:35, Matthew Brush wrote: On 11/08/2011 01:21 PM, Lex Trotman wrote: On Wed, Nov 9, 2011 at 3:40 AM, Nick Treleaven nick.trelea...@btinternet.com wrote: Breaking up logical tasks into functions is crucial to writing maintainable code, functions *are not just about code reuse

Re: [Geany-devel] Proposal: Project settings split - adding geany features

2011-11-08 Thread Nick Treleaven
On 04/11/2011 00:21, Lex Trotman wrote: Just one note that the point of raising it in the ML before doing it was to see if anyone had better ideas for a solution. Geany needs more of that, as exhausting as it may be, rather than taking the first suggestion or patch or pre-implemented pull

[Geany-devel] Short function factoring

2011-11-08 Thread Nick Treleaven
On 04/11/2011 00:21, Lex Trotman wrote: Geany has many places where a short function then calls another short function which calls another short function, none of which are re-used. Personally I find this way of writing code less efficient and very hard to follow and understand as a whole, but

Re: [Geany-devel] Stub project files for sharing

2011-11-08 Thread Nick Treleaven
On 04/11/2011 21:31, Lex Trotman wrote: [...] I see that 'based on' could be useful for making subprojects, and could be added to my proposal. As I see it, this is the same as your proposal, just that we have an explicit pointer from the project file to the stub project file so they don't

Re: [Geany-devel] Stub project files for sharing

2011-11-04 Thread Nick Treleaven
(There was no reply-to list address set for Thomas's mail (weird), so I'm now sending this to the list - possibly a Thunderbird issue). On 04/11/2011 09:56, Thomas Martitz wrote: Am 03.11.2011 15:45, schrieb Nick Treleaven: My solution: A foo.geanystub project file goes in version control

Re: [Geany-devel] Stub project files for sharing - creation plugin

2011-11-04 Thread Nick Treleaven
On 04/11/2011 15:22, Nick Treleaven wrote: That solution seems awkward to me. Especially the creation part where you need to copy, then hand-edit. Normally you don't need to touch such project files at all, but you require not only that, but also that the people know which parts to take out

Re: [Geany-devel] Make document_save_file() show the Save As dialog when necessary - autosave

2011-11-03 Thread Nick Treleaven
On 02/11/2011 23:22, Lex Trotman wrote: Hi Nick, Good idea, I have one question below. [...] It's possible plugins may need updating, but after a quick grep it seems perhaps only the core Save Actions plugin. For auto-saving it may want to ignore documents without an absolute path to avoid

Re: [Geany-devel] Proposal: Project settings split - adding geany features

2011-11-03 Thread Nick Treleaven
On 02/11/2011 23:03, Lex Trotman wrote: [...] Everything else is shared via VCS, so its annoying that one thing isn't. As I said, you can already share settings, you just can't easily synchronize them *if* they change. Whats the problem with doing it? It adds code complexity, plus

[Geany-devel] Stub project files for sharing

2011-11-03 Thread Nick Treleaven
Hi, I think I can improve on Lex's recent proposal to split up project settings into two files, one for version control and one for local changes. The problems with that are: * How do we decide which settings are in each - e.g. build menu commands - a project might want to override the make

Re: [Geany-devel] Proposal: Project settings split

2011-11-02 Thread Nick Treleaven
On 01/11/2011 22:24, Lex Trotman wrote: On 2 November 2011 00:09, Nick Treleavennick.trelea...@btinternet.com wrote: On 31/10/2011 23:50, Lex Trotman wrote: IIUC the functionality would be the same but with different storage? If so I'm not sure this would be worthwhile complexity. A

Re: [Geany-devel] Proposal: Project settings split

2011-11-02 Thread Nick Treleaven
On 02/11/2011 12:10, Lex Trotman wrote: [...] Yes, I just question whether it's worth supporting that. Well, it comes down to how groups and individuals work, you may not want to share projects settings, but I am part of groups who do. Everything else is shared via VCS, so its annoying that

[Geany-devel] Make document_save_file() show the Save As dialog when necessary

2011-11-02 Thread Nick Treleaven
for the same reason was not reliable enough. That code can now be removed, but won't cause users any problems if left. On 02/11/2011 15:27, Nick Treleaven wrote: Branch: refs/heads/master Home: https://github.com/geany/geany Commit: de559ef5d4150e2485ef3ffb865e9c9d3249bcd8 https

Re: [Geany-devel] Make document_save_file() show the Save As dialog when necessary

2011-11-02 Thread Nick Treleaven
On 02/11/2011 15:44, Nick Treleaven wrote: Remembering to show the Save As dialog in all code for the same reason was not reliable enough. That code can now be removed, but won't cause users any problems if left. BTW the only plugin that seems to use dialogs_show_save_as() is geanysendmail

Re: [Geany-devel] Proposal: Project settings split

2011-11-01 Thread Nick Treleaven
On 31/10/2011 23:50, Lex Trotman wrote: Hi All, One area of Geany has been annoying me for some time. At the moment project settings mix session related settings and truely project settings. The session settings are really user specific whilst the project settings are project related. This

Re: [Geany-devel] GIT commit mails format

2011-10-31 Thread Nick Treleaven
On 25/10/2011 00:33, Lex Trotman wrote: [...] If we go back to individual commit mails with diffs, is there anyway to limit how many individual messages get sent out at once and how many lines can be in a diff? Ex. if we merge the gtkbuilder branch at some point, is there anyway to prevent

Re: [Geany-devel] GIT commit mails format

2011-10-31 Thread Nick Treleaven
On 31/10/2011 13:57, Matthew Brush wrote: I'm not sure we would see a flood on a merge, probably only one commit saying 'merged gtkbuilder branch'? The gtkbuilder commits should appear as they are committed, not on merge. I think when you merge it makes all the commits onto the branch that

Re: [Geany-devel] GIT commit mails format

2011-10-31 Thread Nick Treleaven
On 31/10/2011 15:07, Nick Treleaven wrote: On 31/10/2011 13:57, Matthew Brush wrote: I'm not sure we would see a flood on a merge, probably only one commit saying 'merged gtkbuilder branch'? The gtkbuilder commits should appear as they are committed, not on merge. I think when you merge

Re: [Geany-devel] gtkbuilder branch and ui_lookup_widget functions

2011-10-31 Thread Nick Treleaven
On 19/10/2011 08:47, Matthew Brush wrote: On 11-10-17 05:22 AM, Nick Treleaven wrote: Hi Matthew, I'm a bit concerned about the changed ui_lookup_widget (and hookup) functions - these are in the plugin API and can be used independently from Glade. (plugin) API function behaviour should

Re: [Geany-devel] [RFC] Geany Plugin Names / Lua Scripting

2011-10-29 Thread Nick Treleaven
On 29/10/2011 06:41, Matthew Brush wrote: Hi all, Is anybody opposed to removing the geany and Geany prefix from the plugins in Geany-Plugins. I mean at least for the directory name in the source tree, README/Site, and PLUGIN_SET_INFO() name? It's always bothered me when I go into *Geany's*

[Geany-devel] project build tab - Re: GTK+ Version Bump to 2.18 - gtkbuilder and 2.16

2011-10-18 Thread Nick Treleaven
On 18/10/2011 00:36, Matthew Brush wrote: On 11-10-17 04:27 PM, Lex Trotman wrote: On 18 October 2011 10:17, Matthew Brushmbr...@codebrainz.ca wrote: I also noticed some issues with defaults in the project dialog where some stuff doesn't have a default and some of the combo boxes have two

[Geany-devel] gtkbuilder branch and ui_lookup_widget functions

2011-10-17 Thread Nick Treleaven
Hi Matthew, I'm a bit concerned about the changed ui_lookup_widget (and hookup) functions - these are in the plugin API and can be used independently from Glade. (plugin) API function behaviour should not normally change, but it seems the owner parameter is now ignored. If we are caching

Re: [Geany-devel] Geany memory behavior suspected memory leak - ID: 3415254 - g_slice

2011-10-17 Thread Nick Treleaven
On 15/10/2011 00:08, Lex Trotman wrote: 1. gslice is good when you allocate lots of things the same size, and you know the size, since that must be supplied to slice_free, but a large portion of our allocations are strings, so we don't remember the size (see for eg tagmanager). Yes, by fixed

[Geany-devel] geanydocument caching - Re: Geany memory behavior suspected memory leak - ID: 3415254

2011-10-17 Thread Nick Treleaven
On 14/10/2011 19:26, Colomban Wendling wrote: 1. each time all files are closed only 200k is returned to the OS, but since the first open uses 22.8Mb but later ones less, Geany isn't leaking all of it, it is being held by the allocators. Geany at least keeps all the GeanyDocuments alive and

Re: [Geany-devel] GRegex - Re: editing big files can be too slow with tag reparsing

2011-10-17 Thread Nick Treleaven
On 17/10/2011 18:18, Colomban Wendling wrote: GRegex is more reliable besides being more powerful, based on PCRE. The Windows version (at least) of GNU regex we had didn't understand Mac line endings for ^, $. Hehe, even better:) Now we should probably port all our regexes usage to GRegex

Re: [Geany-devel] Geany memory behavior suspected memory leak - ID: 3415254 - g_slice

2011-10-14 Thread Nick Treleaven
On 14/10/2011 04:09, Lex Trotman wrote: Hi All, I have been investigating bug report suspected memory leak - ID: 3415254 http://sourceforge.net/tracker/?func=detailatid=787791aid=3415254group_id=153444 1. each time all files are closed only 200k is returned to the OS, but since the first

Re: [Geany-devel] GTK+ Version Bump to 2.18 - gtkbuilder and 2.16

2011-10-14 Thread Nick Treleaven
On 13/10/2011 12:53, Nick Treleaven wrote: On 13/10/2011 00:55, Matthew Brush wrote: So I would go for 2.16 overall if this brings us Glade 3 support. It should, but I haven't thoroughly tested it with GTK+ 2.16 yet. Did you try the gtkbuilder branch yet on your 2.16 install by any chance

[Geany-devel] per-project prefs - Re: Remove extra whitespace at end of lines in all source files?

2011-10-14 Thread Nick Treleaven
On 13/10/2011 22:24, Matthew Brush wrote: On 10/13/2011 04:42 AM, Nick Treleaven wrote: The stripping feature in Geany should be made a project preference, I don't want to cause noise on non-Geany files. This is actually on my TODO list, unless anyone feels like doing it before me

Re: [Geany-devel] GTK+ Version Bump to 2.18 - gtkbuilder and 2.16 - on_term_font_set

2011-10-14 Thread Nick Treleaven
On 14/10/2011 15:38, Nick Treleaven wrote: I did get a warning: Gtk WARNING: Could not find signal handler 'on_term_font_set' This is because on_term_font_set is not defined unless VTE support is compiled in (on Windows it's disabled

Re: [Geany-devel] Remove extra whitespace at end of lines in all source files?

2011-10-13 Thread Nick Treleaven
On 12/10/2011 23:50, Matthew Brush wrote: On 11-10-12 07:14 AM, Nick Treleaven wrote: On 12/10/2011 06:00, Matthew Brush wrote: Remove extra whitespace at end of lines in all source files. * Processed with rstrip-whitespace.py script added to scripts/ directory. * Script run on all .c and .h

Re: [Geany-devel] GTK+ Version Bump to 2.18

2011-10-13 Thread Nick Treleaven
On 13/10/2011 00:55, Matthew Brush wrote: On 11-10-12 07:04 AM, Nick Treleaven wrote: On 12/10/2011 02:58, Matthew Brush wrote: I've also been working on getting rid of some of the uses of sealed members (ex. widget-window as opposed to gtk_widget_get_window(widget)). It seems many

[Geany-devel] Remove extra whitespace at end of lines in all source files?

2011-10-12 Thread Nick Treleaven
On 12/10/2011 06:00, Matthew Brush wrote: Remove extra whitespace at end of lines in all source files. * Processed with rstrip-whitespace.py script added to scripts/ directory. * Script run on all .c and .h files in src/ and plugins/ directories. * Also remove more than one newline at the

Re: [Geany-devel] Geany is on Github

2011-10-07 Thread Nick Treleaven
On 07/10/2011 16:28, Matthew Brush wrote: Just to start, because I had their Github usernames, I've added Enrico and Colomban as owners, as well as myself since I need access to set the stuff up. Whoever else should be an Owner (Frank, Nick, etc.) just let me (or other Owners) know. Read below

Re: [Geany-devel] Geany is on Github

2011-10-07 Thread Nick Treleaven
On 07/10/2011 17:55, Thomas Martitz wrote: Now, what we could do, for example, is to create another Team called Contributors or something, and this Team could have Pull only access (even without being in a team, anyone can still clone/pull). Myself, I would probably go into the Contributors

Re: [Geany-devel] CTags with GRegex - Re: editing big files can be too slow with tag reparsing

2011-10-03 Thread Nick Treleaven
On 02/10/2011 20:16, Jiří Techet wrote: On Sun, Oct 2, 2011 at 16:34, Nick Treleaven Now done. This fixes my performance issue with doc/geany.html. Needs testing though. Regex parsers are: actionscript cobol html php R Link: http://geany.svn.sourceforge.net/viewvc/geany?view=revisionrevision

Re: [Geany-devel] CTags with GRegex - Re: editing big files can be too slow with tag reparsing

2011-10-02 Thread Nick Treleaven
On 30/09/2011 17:32, Nick Treleaven wrote: On 30/09/2011 14:37, Nick Treleaven wrote: On 27/09/2011 13:17, Nick Treleaven wrote: On 26/09/2011 23:00, Colomban Wendling wrote: Le 26/09/2011 13:48, Nick Treleaven a écrit : I guess the reason is the regex code that is really old which is used

Re: [Geany-devel] Breaks not fixes : SF.net SVN: geany:[5971] trunk/src/socket.c

2011-09-30 Thread Nick Treleaven
On 30/09/2011 02:32, Lex Trotman wrote: Modified: trunk/src/socket.c === --- trunk/src/socket.c 2011-09-29 17:16:58 UTC (rev 5970) +++ trunk/src/socket.c 2011-09-29 17:49:42 UTC (rev 5971) @@ -588,7 +588,7 @@ gint fd,

[Geany-devel] CTags with GRegex - Re: editing big files can be too slow with tag reparsing

2011-09-30 Thread Nick Treleaven
On 27/09/2011 13:17, Nick Treleaven wrote: On 26/09/2011 23:00, Colomban Wendling wrote: Le 26/09/2011 13:48, Nick Treleaven a écrit : I'm reduced to using Windows. I guess the reason is the regex code that is really old which is used for Windows builds. Maybe we could use GRegex now we have

Re: [Geany-devel] CTags with GRegex - Re: editing big files can be too slow with tag reparsing

2011-09-30 Thread Nick Treleaven
On 30/09/2011 14:37, Nick Treleaven wrote: On 27/09/2011 13:17, Nick Treleaven wrote: On 26/09/2011 23:00, Colomban Wendling wrote: Le 26/09/2011 13:48, Nick Treleaven a écrit : I'm reduced to using Windows. I guess the reason is the regex code that is really old which is used for Windows

Re: [Geany-devel] Bug on utils_string_replace()?

2011-09-29 Thread Nick Treleaven
On 28/09/2011 20:56, Colomban Wendling wrote: Le 28/09/2011 19:36, Frank Lanitz a écrit : On Sun, 25 Sep 2011 20:11:06 +0200 [...] Just want to bring it up again. Release is near so at least we should add a comment to api documentation about this. Right. Fixed now in r5965, thanks. If we

Re: [Geany-devel] editing big files can be too slow with tag reparsing

2011-09-28 Thread Nick Treleaven
/2011 17:17, Nick Treleaven wrote: 1. Very big files x MB - I don't think we need to support that case specially, Geany is a programmer's editor. The user can just open as filetype none if they only want basic editing. In the open dialog, choose filetype None and you can easily open big files

Re: [Geany-devel] editing big files can be too slow with tag reparsing

2011-09-28 Thread Nick Treleaven
On 28/09/2011 12:03, Nick Treleaven wrote: @Nick, there are lots of large generated files a programmer might want to check, especially XMLs and logs, saying there is an xxxGb maximum is wrong, it depends on the machine the user has. Forcing them to have to go use another editor or to not have

Re: [Geany-devel] editing big files can be too slow with tag reparsing - HTML

2011-09-27 Thread Nick Treleaven
On 27/09/2011 01:53, Colomban Wendling wrote: Le 27/09/2011 01:44, Lex Trotman a écrit : [...] Pentium 4, 1.9 GHz, 512 MB ram. I'm no windows expert, but isn't 512mb considered kinda marginal, 1gb better. Whats your general windows performance like? I doubt that, even on Windows, it

Re: [Geany-devel] editing big files can be too slow with tag reparsing - HTML

2011-09-27 Thread Nick Treleaven
On 26/09/2011 23:00, Colomban Wendling wrote: Le 26/09/2011 13:48, Nick Treleaven a écrit : I'm reduced to using Windows. I guess the reason is the regex code that is really old which is used for Windows builds. Maybe yeah, but I find it a little weird you see *so much* slowness

Re: [Geany-devel] editing big files can be too slow with tag reparsing

2011-09-27 Thread Nick Treleaven
On 26/09/2011 22:38, Colomban Wendling wrote: So, I'm soliciting your impressions, opinions, remarks, etc. Also, is this whole thing important enough to break string freeze less than a week before release? (read: to have some strings untranslated) Well, it's not just untranslated strings -

Re: [Geany-devel] Hi Guys

2011-09-27 Thread Nick Treleaven
Hi, On 27/09/2011 16:44, Jacques du Rand wrote: Quick Question: -I get a copy of source via svn compiles fine -Builds fine -Run it asgeany-svn-snapshotsrc/geany -c ~/.config/geanydev THEN : ERROR1: I can't see any open files in File Open Dialog I can see files in Project - Open dailog ?

Re: [Geany-devel] editing big files can be too slow with tag reparsing

2011-09-27 Thread Nick Treleaven
On 26/09/2011 22:38, Colomban Wendling wrote: Le 24/09/2011 13:32, Lex Trotman a écrit : [...] How about automagically disabling auto-reparsing per-file if it takes too long (1s?) at runtime? Without needing to change the setting that is. I disagree, it is a bad idea to turn off a setting

Re: [Geany-devel] editing big files can be too slow with tag reparsing - HTML

2011-09-26 Thread Nick Treleaven
On 24/09/2011 11:59, Colomban Wendling wrote: Le 22/09/2011 18:30, Nick Treleaven a écrit : Hi, Whilst opening doc/geany.html I found it takes5s on my machine to load. It's a big document though, still perhaps the HTML tag parser performance could be improved. I think 5s is weird in the first

Re: [Geany-devel] TODO: update GTK+ version in Hacking

2011-09-22 Thread Nick Treleaven
On 21/09/2011 23:06, Matthew Brush wrote: I noticed that this section[1] in the hacking manual mentions: This is because Geany depends on GTK 2.8. API symbols from newer GTK/GLib versions should be avoided to keep the source code building against GTK 2.8. Now fixed. I guess this should be

[Geany-devel] editing big files can be too slow with tag reparsing

2011-09-22 Thread Nick Treleaven
Hi, Whilst opening doc/geany.html I found it takes 5s on my machine to load. It's a big document though, still perhaps the HTML tag parser performance could be improved. But now we have tag reparsing, editing geany.html becomes painful. I know we can disable reparsing, but what else could we

Re: [Geany-devel] SF.net SVN: geany:[5937] trunk - use_atomic_file_saving

2011-09-21 Thread Nick Treleaven
On 20/09/2011 23:30, Lex Trotman wrote: Rename use_safe_file_saving various pref to use_atomic_file_saving as it's not 'safe'. Use old pref if new one is missing for compatibility with Geany= 0.20. Hi Nick, I like atomic, large mushroom cloud when you use this setting:) Hehe Don't

[Geany-devel] geany-commits reply subject line

2011-09-21 Thread Nick Treleaven
Hi all, When replying to geany-commits mails, please edit the subject to include something descriptive, not just the revison number. That would really help pick out any important needles in the haystack of emails ;-) Thanks, Nick ___ Geany-devel

Re: [Geany-devel] Various pref changes not ignored on dialog cancel

2011-09-21 Thread Nick Treleaven
On 16/09/2011 18:11, Dimitar Zhekov wrote: Attached is the stash_tree_display(). I thought that a row changed signal will be needed, but it works without it. The renreders always reget their data when the tree is shown. No string changes. _append_pref() ignores the new action argument.

Re: [Geany-devel] Various pref changes not ignored on dialog cancel

2011-09-21 Thread Nick Treleaven
On 21/09/2011 16:31, Nick Treleaven wrote: On 16/09/2011 18:11, Dimitar Zhekov wrote: Attached is the stash_tree_display(). I thought that a row changed signal will be needed, but it works without it. The renreders always reget their data when the tree is shown. No string changes. _append_pref

Re: [Geany-devel] Various pref changes not ignored on dialog cancel

2011-09-16 Thread Nick Treleaven
--- On Thu, 15/9/11, Colomban Wendling lists@herbesfolles.org wrote: How about adding a PREF_CANCEL action to prefs.c only that will call a stash_group_cancel() to reset the values? The other I don't think that's a good design, the stash group settings should not actually be changed

Re: [Geany-devel] various prefs warning use_safe_file_saving

2011-09-16 Thread Nick Treleaven
--- On Thu, 15/9/11, Colomban Wendling lists@herbesfolles.org wrote: The dangerous settings are gio_unsafe_save_backup, use_gio_unsafe_file_saving and use_safe_file_saving, and IMHO their names are quite indicative. ... I agree, I like Dimitar's feature. I don't want to make it

Re: [Geany-devel] various prefs warning - changing 'various'

2011-09-16 Thread Nick Treleaven
--- On Fri, 16/9/11, Matthew Brush mbr...@codebrainz.ca wrote: I agree, I like Dimitar's feature. I don't want to make it hard to use, just more obvious that the user should be careful. One solution would be to just rename Various to Advanced which implies that these are not your regular

  1   2   3   4   5   >