2008/2/6, Sebastien Bacher <email address hidden>:
> Could you mail the nautilus-list to get their comments on the change?
Hello dear Nautilus masters :-)
I have recently opened a bug on Launchpad and Gnome Bug tracker which
is more a code organization issue than an improvement.
Here is the description of the bug :
<quote>
Hello,
I wanted to patch nautilus-action to add submenu support to it and I
noticed that the struct _NautilusMenuItemDetails is not available in any
of the nautilus extension headers. A search on the web showed me that it
has been defined in the nautilus-menu-item.c source file. So we need to
move it from this file to the header file. Otherwise it's not possible
to use this struct in an external code.
I have already made a patch.
Best Regards
</quote>
So basically, Sebastien wanted to know what you think about this bug
and patch before applying it to Ubuntu.
I have attached the same patch to the Gnome Bug .
So basically the patch is just a move of a struct out of the source
*.c file to the header.
There is another struct that is defined in that source file but that
is commented as Private in the header, so I didn't move it.
Longing for your answer ...
Best Regards
Aziz
> --
> NautilusMenuItemDetails not defined in header file
> https://bugs.launchpad.net/bugs/188288
> You received this bug notification because you are a direct subscriber
> of the bug.
>
2008/2/6, Sebastien Bacher <email address hidden>:
> Could you mail the nautilus-list to get their comments on the change?
Hello dear Nautilus masters :-)
I have recently opened a bug on Launchpad and Gnome Bug tracker which
is more a code organization issue than an improvement.
Here is the description of the bug :
<quote>
Hello,
I wanted to patch nautilus-action to add submenu support to it and I emDetails is not available in any menu-item. c source file. So we need to
noticed that the struct _NautilusMenuIt
of the nautilus extension headers. A search on the web showed me that it
has been defined in the nautilus-
move it from this file to the header file. Otherwise it's not possible
to use this struct in an external code.
I have already made a patch.
Best Regards
</quote>
So basically, Sebastien wanted to know what you think about this bug
and patch before applying it to Ubuntu.
I have attached the same patch to the Gnome Bug .
Gnome bug : http:// bugzilla. gnome.org/ show_bug. cgi?id= 513854 /bugs.launchpad .net/bugs/ 188288
Launchpad bug : https:/
So basically the patch is just a move of a struct out of the source
*.c file to the header.
There is another struct that is defined in that source file but that
is commented as Private in the header, so I didn't move it.
Longing for your answer ...
Best Regards
Aziz
> -- mDetails not defined in header file /bugs.launchpad .net/bugs/ 188288
> NautilusMenuIte
> https:/
> You received this bug notification because you are a direct subscriber
> of the bug.
>