Hi! There are 54 (!) unmerged PR's including some of mine.
PRs are not automatically merged.
This is 1-year PR which is absolutely ready but still umnerged: https://github.com/geany/geany/pull/172/files
Dunno, but personally I don't like non-standard mouse button functionality so I won't merge it, and I guess nobody else sees the point either.
This commit looks ok to me, if nobody else complains or beats me to it I will commit when I get a chance during the week
Still nothing: https://github.com/geany/geany/pull/176
My last comment noted that you have added a setting, but not documented it. AFAICT that is still the case.
Fixed memry leaks unmerged: https://github.com/geany/geany/pull/186/files
This change is not explained, what does it do, why do you want to change it?
Cheers Lex
-- Best regards, Pavel Roschin aka RPG _______________________________________________ Devel mailing list Devel@lists.geany.org https://lists.geany.org/cgi-bin/mailman/listinfo/devel
Dunno, but personally I don't like non-standard mouse button functionality so I won't merge it, and I guess nobody else sees the point either.
There was a point from Ban:
We do it unconditionally for notebook tabs, and AFAIK there is no common behavior on middle click in this case, so closing wouldn't be unexpected, would it?
So I removed ability to make it configurable. I don't see the reason to make whole plugin because of one mouse button and simplest patch but who should decide - close or merge?
My last comment noted that you have added a setting, but not documented it. AFAICT that is still the case.
This patch also involves doc/geany.txt. If doc/geany.txt is not documentation, what is? https://github.com/geany/geany/pull/176/files
Fixed memry leaks unmerged: https://github.com/geany/geany/pull/186/files
This change is not explained, what does it do, why do you want to change it?
I cannot leave bug report on SF because of buggy accounts and cannot open Issue in GitHub, so I decided to make PR. This PR eliminates two memory leaks: * g_listenv returns an array of strings and should be freed, g_return* breaks this * after returning GString's buffer->str, buffer should be freed but it doesn't (even if it is a static pointer it will make a noise in valgrind report).
If you want to keep them just close PR. I already closed other old PR's I have no time to maintain at this moment. But these seems to be clear and simple.
-- Best regards, Pavel Roschin aka RPG