On Mon, 2005-10-17 at 21:17 +0200, Christian Neumair wrote:
Am Montag, den 17.10.2005, 10:20 +0200 schrieb Alexander Larsson:
On Fri, 2005-10-14 at 16:51 +0200, Christian Neumair wrote:
The attached patches are meant to fix cancellation of URI activations
through the panel. They are against libgnome, gnome-panel and gnome-vfs,
and break string freeze, and potentially even API freeze. I wonder
whether adding an element to a public enum is consiered an API change.
Life would have been easier if GnomeURLError used negative error codes,
reserving all = 0 for GnomeVFSResult retvals.
GnomeURLError was written before gnome_vfs_show_url existed...
I've also CCed random people that often occur in recent ChangeLogs of
the affected producs and/or released tarballs. MAINTAINERS files aren't
up to date for any of these.
The gnome-vfs patch should really modify _gnome_vfs_get_slow_mime_type()
(it is the only used for this function) instead of copying it into the
function.
Yes, I know. I just didnt want to change that header. You reminded me
that even private API deserves some attention. Attaching a new patch
set.
What do you propose to resolve the
gnome_vfs_get_mime_type/_gnome_vfs_get_slow_mime_type signature
inconsistency the change introduces, considering that
_gnome_vfs_get_slow_mime_type is very likely to become public one day,
if not for GnomeVFS 2.14?
One could expose the public version of the function which is similar to
gnome_vfs_get_mime_type, which just calls the current (patched)
_gnome_vfs_get_slow_mime_type. That makes the public interface similar,
and they are just helper functions anyway, so the slight limitation
doesn't really hurt much.
=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=
Alexander LarssonRed Hat, Inc
[EMAIL PROTECTED][EMAIL PROTECTED]
He's a maverick flyboy master criminal in a wheelchair. She's a warm-hearted
renegade doctor with a song in her heart and a spring in her step. They fight
crime!
___
gnome-i18n mailing list
gnome-i18n@gnome.org
http://mail.gnome.org/mailman/listinfo/gnome-i18n
