Re: [PATCH] - Add drag drop of bookmarks to places sidebar

From: Jamie McCracken <jamiemcc blueyonder co uk>
To: Alexander Larsson <alexl redhat com>
Cc: "nautilus-list gnome org" <nautilus-list gnome org>
Subject: Re: [PATCH] - Add drag drop of bookmarks to places sidebar
Date: Wed, 13 Jul 2005 10:55:55 +0100

Alexander Larsson wrote:

I don't like the name "user data" for something that only contains
volumes. It would be better to just call it volumes. That way you can
correctly own the volume to, and don't have to use the unref_volumes
hack before gtk_list_store_clear and other places.

its called user_data cause it will either contain a volume or a bookmarkpointer. I need it to get at the underlying object for the popup contextmenu which im currently implementing. Is that okay then?

Can you clarify what you mean by correctly owning the volume? Wont Istill have to manually unref it prior to clearing the list store?


--
Mr Jamie McCracken
http://www.advogato.org/person/jamiemcc/

Follow-Ups:
- Re: [PATCH] - Add drag drop of bookmarks to places sidebar
  - From: Alexander Larsson

References:
- [PATCH] - Add drag drop of bookmarks to places sidebar
  - From: Jamie McCracken
- Re: [PATCH] - Add drag drop of bookmarks to places sidebar
  - From: Jamie McCracken
- Re: [PATCH] - Add drag drop of bookmarks to places sidebar
  - From: Alexander Larsson

[Date Prev][Date Next] [Thread Prev][Thread Next] [Thread Index] [Date Index] [Author Index]