[Github-comments] [geany/geany] Gi signals (#1038)

Thomas Martitz notifications at xxxxx
Thu Jun 23 20:17:45 UTC 2016


> @@ -3250,6 +3250,7 @@ void scintilla_release_resources(void) {
>  static void *copy_(void *src) { return src; }
>  static void free_(void *doc) { }
>  
> +GEANY_API_SYMBOL
>  GType scnotification_get_type(void) {
>  	static gsize type_id = 0;

>     Anyway the discussion out of scope for this PR
>
> Code review is out of scope for this PR? At the least it could do like
> the code @b4n <https://github.com/b4n> linked to, and avoid ugly (and
> useless) casts.
>

Yes, reviewing *this* code is out of scope for *this* PR. The code you 
question is not changed in this PR and is already part Geany.

This PR just adds the GEANY_API_SYMBOL decoration and the implementation 
is irrelevant to the PR.


---
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/1038/files/6b5a47d05f575b321ff158044420864ee9e59889#r68306942
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20160623/e0a7c8fe/attachment.html>


More information about the Github-comments mailing list