[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Gnash-dev] gtkglext, et al
From: |
strk |
Subject: |
Re: [Gnash-dev] gtkglext, et al |
Date: |
Sun, 23 Apr 2006 02:21:01 +0200 |
On Sat, Apr 22, 2006 at 06:32:29PM +0200, Bastiaan Jacques wrote:
> On Saturday 22 April 2006 16:44, Rob Savoye wrote:
> > As you dig deeper, you'll see lots of ugly code
> > using structs instead of classes, and other non C++ things. I've been
> > trying to change things to be more real C++ when I can.
>
> I've done this for a little class in Movie.h. Would you take a look at
> the attached patch and let me know if that's how you'd like to see things
> done in the future? (And of course you can commit it, if you like.)
I've given a quick read of your patch. It surely looks more C++ then
it is, but maybe, if the import_info is really just an helper class
of Movie as comments say, it could be an all-privates with Movie as a friend.
What do others think ?
--strk;
- [Gnash-dev] gtkglext, et al, Bastiaan Jacques, 2006/04/22
- Re: [Gnash-dev] gtkglext, et al, Rob Savoye, 2006/04/22
- Re: [Gnash-dev] gtkglext, et al, Bastiaan Jacques, 2006/04/22
- Re: [Gnash-dev] gtkglext, et al, Rob Savoye, 2006/04/22
- Re: [Gnash-dev] gtkglext, et al, Bastiaan Jacques, 2006/04/22
- Re: [Gnash-dev] gtkglext, et al, Rob Savoye, 2006/04/22
- Re: [Gnash-dev] gtkglext, et al, Bastiaan Jacques, 2006/04/22
Re: [Gnash-dev] gtkglext, et al, Bastiaan Jacques, 2006/04/22
- Re: [Gnash-dev] gtkglext, et al,
strk <=
- Re: [Gnash-dev] gtkglext, et al, Bastiaan Jacques, 2006/04/22
- Re: [Gnash-dev] gtkglext, et al, strk, 2006/04/26
- Re: [Gnash-dev] gtkglext, et al, Bastiaan Jacques, 2006/04/26
- Re: [Gnash-dev] gtkglext, et al, strk, 2006/04/26