Hi there,<br><br>It's me again :-) I just have some questions about committing on Sourceforge. Is there any convention into messages ? May we commit as much as we want ? Are there some (strict) limitations ? Probably I'll only commit files for my plugin but...<br>
<br>Thanks & regards,<br>Cedric<br><br>ps. I can now compile using the automated build tool :-) great !!!<br><br><div class="gmail_quote">2009/8/10 Enrico Tröger <span dir="ltr"><<a href="mailto:enrico.troeger@uvena.de">enrico.troeger@uvena.de</a>></span><br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">On Tue, 11 Aug 2009 04:26:51 +0800, Chow wrote:<br>
<br>
Hey,<br>
<br>
<br>
attached is a little patch which cleans up the includes and improves<br>
ordering of the included header files.<br>
The main change is to include "geany.h" very early, this header will<br>
implicitly include most of the GLib and GTK headers you need. So, there<br>
is no need to explicitly include something like <glib/gmacros.h> which<br>
is deprecated due to GTK's single header policy anyway. I don't want to<br>
bore you with details, just make "geany.h" your first include in<br>
general should do the trick.<br>
<br>
In the future, i.e. once Geany 0.18 is released, things will get a lot<br>
easier, then you only need to include "geanyplugin.h" which includes<br>
most other necessary Geany and GTK headers.<br>
The API docs, especially the howto describe this issue as well, IIRC.<br>
<div class="im"><br>
<br>
>As for the issue with the for loop, the reason for that happening is<br>
>because gcc defaults to -std=gnu89, which is based on the C90 standard,<br>
>whereas declarations within the for loop only work with -std=c99 and<br>
>above. I think it's best to stick to this standard (C90) for<br>
>portability reasons.<br>
<br>
</div>While there are probably not many people out there (anymore) who use a<br>
C89/C90 compiler, there are at least a few of them. And so we try to<br>
keep the Geany code compiling with gcc -ansi, i.e. compatible with C89.<br>
Nobody is forced to do the same with plugins, but it just makes it<br>
easier for users. The current code still doesn't compile with a pure<br>
C89 compiler because C89 doesn't allow to mix code and variable<br>
declarations. Cédric, it's completely up to you whether you want to<br>
"fix" this or just keep it as it is.<br>
<br>
If you want to see the warnings I'm talking about, just check<br>
<a href="http://nightly.geany.org/misc/build_gtk28_plugins_stderr.log" target="_blank">http://nightly.geany.org/misc/build_gtk28_plugins_stderr.log</a> tomorrow<br>
(after August 11, 04:00 UTC). There should appear a few warnings. But<br>
again, it's completely up to you whether to remove them or to keep them.<br>
<div><div></div><div class="h5"><br>
<br>
Regards,<br>
Enrico<br>
<br>
--<br>
Get my GPG key from <a href="http://www.uvena.de/pub.asc" target="_blank">http://www.uvena.de/pub.asc</a><br>
</div></div><br>_______________________________________________<br>
Geany-devel mailing list<br>
<a href="mailto:Geany-devel@uvena.de">Geany-devel@uvena.de</a><br>
<a href="http://lists.uvena.de/cgi-bin/mailman/listinfo/geany-devel" target="_blank">http://lists.uvena.de/cgi-bin/mailman/listinfo/geany-devel</a><br>
<br></blockquote></div><br>