On 31 October 2015 at 20:39, Abel akronix5@gmail.com wrote:
I understand. I'm trying to solve the inconsistency of being editing a document on split window but having the status bar showing something else (Should I open a issue to report this bug?). So, yes, you can take this as a request for the API :)
Umm, yeah, that sounds like a bug. And then you have something to refer to when you submit the PR with the changes.
I haven't ever doxygened anything. But would it be enough for getting added it to the API just adding doxygen comments to it?
The documentation is added by adding doxygen comments for the function, see doxygen docs and examples of other API functions. Generating the docs is in HACKING IIRC.
I want to test that the function is suitable for the purpose I want it for, before sending a PR for getting it into the API.
Well, you can make and test the changes in your git clone first, and I think so long as its a separate commit, the changes to make the function available can be part of the PR that needs it. That way there are no dependencies between two PRs, one making the function available and one using it.
Cheers Lex
Saludos,
Abel.On 31 October 2015 at 11:28, Lex Trotman elextr@gmail.com wrote:
Nothing is usually added to the API "just because" since it has to be maintained and supported and remain stable, and if the particular function is too deep in the implementation that may constrain later changes to Geany. Things are mostly added these days if a plugin has a need.
I guess we can take this as a request for the function to be added to the API :)
Maybe you can make a PR to speed the process. Note the need to doxygen document the function.
Cheers Lex
On 31 October 2015 at 19:43, Abel akronix5@gmail.com wrote:
And why is not such function in the API? It's very used across all the geany code and there's no reason to think a plugin wouldn't ever want to use it.
I want to display the document statistics of the document opened on split window when it gets the focus. ui_set_statusbar() let you display text but doesn't give you access to create_statusbar_statistics() neither to DEFAULT_STATUSBAR_TEMPLATE.
Saludos,
Abel.On 31 October 2015 at 03:59, Lex Trotman elextr@gmail.com wrote:
See the big warning message at the start of the plugins doco http://www.geany.org/manual/reference/ "Do not use any symbol not in the documentation".
On more recent versions of Geany the restriction on use of functions not in the API is better enforced to reduce the risk of accidental use. Functions not in the API never worked on Windows I believe, so a plugin that used non-API functions was not portable as well as possibly breaking without warning.
Can you not use
http://www.geany.org/manual/reference/ui__utils_8h.html#aa0948006d2f45a2a2a6...
Cheers Lex
On 31 October 2015 at 12:47, Abel akronix5@gmail.com wrote:
Hi!
I'm hacking split window plugin and I wanted to use the function ui_update_statusbar(), located in ui_utils.h
When I try to compile I get the following warning:
"warning: implicit declaration of function 'ui_update_statusbar' [-Wimplicit-function-declaration]"
And if I try yo execute the bin generated it doesn't allow you to enable that plugin, it is never shown anymore.
The plugin includes geanyplugin.h which in turn includes ui_utils.h
Some help or hints?
Devel mailing list Devel@lists.geany.org https://lists.geany.org/cgi-bin/mailman/listinfo/devel
Devel mailing list Devel@lists.geany.org https://lists.geany.org/cgi-bin/mailman/listinfo/devel
Devel mailing list Devel@lists.geany.org https://lists.geany.org/cgi-bin/mailman/listinfo/devel
Devel mailing list Devel@lists.geany.org https://lists.geany.org/cgi-bin/mailman/listinfo/devel
Devel mailing list Devel@lists.geany.org https://lists.geany.org/cgi-bin/mailman/listinfo/devel