Since I have $EDITOR set to Geany I don't know what software will call it with what filenames. One piece of software I use occasionally uses filename:nn as a temp file name & I have to change name each time :-(. So I have some minor interest in fixing it (but use it rarely enough that I hadn't even filed a bug yet ;-).
I think its not a good idea as the default since Geany can be used by other software (eg filemanagers) to open/create files with any name.
If the file exists, that's not a problem, neither now nor with my patch, since Geany will detect that it exists and open it as-is, without trying to strip the line/columns.
Understood that, no problem here, its a definite improvement.
And with my patch it's even better, since if the file to be opened at a given line/column doesn't exist, it forgets the line and columns and opens the full path as given -- no line/column stripping.
Thats ok too.
I think that the only problem users can see with the behavior of the patch is if they have the files "foo:0" and "foo:0:1" and they want to open "foo:0" at line 1 with the :nn syntax.
What about if you have "file" and you want to create "file:nn" as a new file (my example above)? IIUC it will instead open "file" at line nn?
I mean, with the patch, Geany can be used transparently with files that have :nn suffixes -- apart using the :nn suffix to open at a given line/column that doesn't always do magic (but almost).
But maybe add a command line option to specify that the filename IS suffixed with :line:col so you can still paste terminal output. Perhaps -a and --at.
Perhaps it's a possibility, yes -- even though I'd prefer to be able to set it as the default.
Just alias ga to geany --at & save even more typing ;-)
Cheers Lex
Cheers, Colomban _______________________________________________ Geany-devel mailing list Geany-devel@uvena.de http://lists.uvena.de/cgi-bin/mailman/listinfo/geany-devel