Re: [[gnomemm] gnome-vfsmm patch]



It looks good. The Glib::wrap() is fine, I think. Please patch the ChangeLog
and add it to bugzilla. Well done.

If you are doing anything new then think about writing more examples too.

Per Kristian Gjermshus <pkg newmad no> wrote:
> I fixed gnome-vfsmm so that my little program works with it. The problem 
> was that gnome_vfs_file_info_new was not called on some FileInfo 
> objects. I also fixed Gnome::Vfs::Uri so that it throws the correct 
> exceptions on failure.
> 
> I have not looked much at the c++-wrappers before, so I might have done 
> something wrong. Is it safe to just do a gnome_vfs_file_info_new and 
> then Glib::Wrap it afterwards? This will not lead to leaks?
> 
>                                                    Per Kristian
> 

> --------------------------------------------- 
>	Attachment: uri-fileinfo.diff 
>	MIME Type: text/plain 
> --------------------------------------------- 


Murray Cumming
murrayc usa net
www.murrayc.com





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