Re: [Github-comments] [geany/geany] Filetype file editing overrides filetype changes with new versions of Geany (#1552)

2017-07-24 Thread elextr
@eliardocosta then make a specific issue, stop hijacking this one.

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

Re: [Github-comments] [geany/geany] Filetype file editing overrides filetype changes with new versions of Geany (#1552)

2017-07-24 Thread Eliardo Costa
Thank you for your reply! I deleted the file but the problem continues :/

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

Re: [Github-comments] [geany/geany] Regression: No syntax highlighting for bash lists with one item (#1308)

2017-07-24 Thread elextr
Closed #1308 via #1503.

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

Re: [Github-comments] [geany/geany] Filetype file editing overrides filetype changes with new versions of Geany (#1552)

2017-07-24 Thread elextr
The edited `filetypes.common` is saved in your user config directory and 
overrides the system file.  Delete the one in the user config.

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

Re: [Github-comments] [geany/geany] Add utils_get_real_path() and use it (#1224)

2017-07-24 Thread Thomas Martitz
What about this?

(On a related note, our `realpath()` implementation for win32, called by 
`tm_get_real_path()` is buggy, see #1534)

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

Re: [Github-comments] [geany/geany] Update Scintilla to version 3.7.5 (#1503)

2017-07-24 Thread Thomas Martitz
I'm using scintilla-375 + the above bugfix without problems thus far.

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

Re: [Github-comments] [geany/geany] Snippets: Use ASCII ellipsis for the cursor placeholder (957b49b)

2017-07-24 Thread Thomas Martitz
I disagree that it looks better, I prefer one char. Unfortunately … is 
troublesome.
I then chose _ because it's also less likely to mess up the tagmanager due to 
syntax errors introduced by the ellipsis.

I don't feel very strong about it, though.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/commit/957b49b868214a4aea81641dedeabb98a497e4c1#commitcomment-23274521

Re: [Github-comments] [geany/geany] Fixes & changes for #1470 (#1554)

2017-07-24 Thread Thomas Martitz
kugel- commented on this pull request.



> @@ -2458,7 +2457,7 @@ void editor_insert_text_block(GeanyEditor *editor, 
> const gchar *text, gint inser
gint line_start = sci_get_line_from_position(sci, insert_pos);
GString *buf;
const gchar *eol = editor_get_eol_char(editor);
-   gint idx;
+   gint idx = 0;

Looking at it more closely, the issue is more serious, and your fix seems 
insufficient. I'll look into it.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/1554#discussion_r129151468

Re: [Github-comments] [geany/geany] Geany Locking up when typing or copying of text (#1540)

2017-07-24 Thread AdamDanischewski
It may seem daunting because of the amount of rows, but it shouldn't take very 
long since its a binary search the complexity is O(log n) at worst. 

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

Re: [Github-comments] [geany/geany] Add an editor option to enable IME's candidate window display inline (#1514)

2017-07-24 Thread elextr
@sinpowei well, somebody (other than you :) needs to test it, but not many 
Geany contributors use IME languages so most (all?) can't test it.

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