[maemo-developers] [maemo-developers] Ignored patches, ignored bugs

[From: Murray Cumming]

On Wed, 2007-01-24 at 13:53 +0200, Marius Vollmer wrote:
> "ext maemo-developers-bounces at maemo.org" <maemo-developers-bounces at maemo.org> writes:
> 
> > Maybe the list of important bugs for hildon-fm is just too long to get
> > the easy patches fixed right now ;) 
> 
> Yep, the C++ bindings are unfortunately pretty low priority for me.
> As far as language bindings go, C++ doesn't need them (people can just
> use the C API from C++).

Yet this shouldn't make it difficult to spend 10 minutes to apply
incredibly simple patches. I know that a 10 minute context-switch can
disturb your more important work, so there should be someone who can get
this done, for my sake.

> Also, my general attitude is that the wrappers should be flexible
> enough to work around buglets in the wrapped code.
> 
> We are talking about this patch to hildon-fm, right?

Among others, yes.

> 
> --- hildon-fm/hildon-file-system-model.h	(revision 1305)
> +++ hildon-fm/hildon-file-system-model.h	(working copy)
> @@ -96,10 +96,12 @@
>  
>  typedef struct _HildonFileSystemModelPrivate HildonFileSystemModelPrivate;
>  
> -typedef struct {
> +typedef struct _HildonFileSystemModel HildonFileSystemModel;
> +
> +typedef struct _HildonFileSystemModel {
>      GObject parent;
>      HildonFileSystemModelPrivate *priv;
> -} HildonFileSystemModel;
> +};
> 
> I'd say that you should fix the wrapper generator to not be so
> brittle, or add some way of hinting it around issues like this.

Yes, we can do that work (and other wrapper generators can do that work
too), or these simple patches can be applied. In general, I don't
complicate the wrapper generator unless absolutely necessary. For
instance, if it's impossible to get the necessary change made to the C
API.

We do have several hints/options for other issues already.

> Nevertheless, I am going to apply the patch.  Eventually. :-)

Thanks. That's the simplest fix.

-- 
Murray Cumming
murrayc at murrayc.com
www.murrayc.com
www.openismus.com