Re: Fixing the Gst::Message classes

[Murray Cumming <murrayc murrayc com>]

On Tue, 2007-12-11 at 21:37 -0500, José Alburquerque wrote:
> The good news is that I was able to implement a Gst::Message::wrap() 
> method and modify gstmm to use it for wrapping messages.  Unfortunately, 
> I think I've reached my C++ knowledge limit :-).  It looks like the 
> Gst::Message::wrap() method should return the appropriate class based on 
> the "message type", however, I tested with the modified 
> test-mini-object-wrap test (to use the new Gst::Message::wrap() method) 
> and also by trying to cast a Gst::Message (wrapped form a 
> Gst::MessageError) to a Gst::MessageError in the on_bus_message() 
> function of the ogg player example (by using RefPtr<...>::cast_dynamic) 
> and both tests were unsuccessful.
> 
> At this point I'm pretty much stuck so I'm hoping you might be able to 
> shed some light on what I'm doing.  Would you be able to look at my 
> changes?  Thanks.

In Gst::Message::wrap() I changed it to just use new rather than
wrap_auto() and reinterpret_cast<> because that seemed simpler. I had to
add constructors for that. It seems to work now (in svn). I added a new
test for this new wrap() function.

I also changed the calls to wrap() to Gst::Message::wrap() to avoid any
confusion.

-- 
murrayc murrayc com
www.murrayc.com
www.openismus.com