[Geany-devel] get build command from a plugin

Lex Trotman elextr at xxxxx
Wed Feb 1 22:34:35 UTC 2012


On Thu, Feb 2, 2012 at 5:10 AM, Dimitar Zhekov <dimitar.zhekov at gmail.com> wrote:
> On Wed, 1 Feb 2012 11:24:01 +1100
> Lex Trotman <elextr at gmail.com> wrote:
>
>> On Wed, Feb 1, 2012 at 5:20 AM, Dimitar Zhekov <dimitar.zhekov at gmail.com> wrote:
>> > On Tue, 31 Jan 2012 15:39:52 +1100
>> >
>> > I compile Geany with -Wall -W -Wno-unused-parameter (not the normal
>> > practice) and received a few warnings:
>>
>> Well, it should be normal practice, thats what HACKING says, I just
>> forgot to set it with a new clean clone, oops.  Actually you should
>> add -O2 because a couple of the warnings need the dataflow computation
>> that the optimisation does.
>
> I only listed the warnings, -O2 is a sine qua non (albeit -Os
> -freorder-blocks is better for some systems).
>
>> The semantics of a command (well actually the label) that is "" vs
>> NULL is important, a "" label hides commands from a lower priority
>> source without showing itself in the menu, a NULL does not. [...]
>>
>> Unfortunately there isn't anthing else besides NULL that is sensible
>> to return to indicate out of range.
>
> So returning "" for COMMAND is possible...
>
>> So I've exposed build_get_group_count() as well so you can get the
>> count and then its your fault if you pass an out of range index :).
>
> ...but a count is fine too. :) Let's hope BuildFuncs makes it into
> the plugin interface, the current lack of build access is weird.

Well, my intention is that when Matthew has tested writing I will add
it if no other developers object in the meantime.

Cheers
Lex

>
> --
> E-gards: Jimmy
> _______________________________________________
> Geany-devel mailing list
> Geany-devel at uvena.de
> https://lists.uvena.de/cgi-bin/mailman/listinfo/geany-devel



More information about the Devel mailing list