[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    Re: kdemultimedia
From:       George Staikos <staikos () kde ! org>
Date:       2003-05-31 21:20:58
[Download RAW message or body]

On Thursday 29 May 2003 19:56, Arnold Krille wrote:
> On Wednesday 28 May 2003 06:52, George Staikos wrote:
> > CVS commit by staikos:
> > more dynamic_cast fixes/notes
> >   M +1 -0      arts/modules/effects/kstereovolumecontrolgui_impl.cpp  
> > 1.3
>
> The line you are asking shouldn't fail since the mapper is in the same
> header and object-file as the effect. But perhaps you could clearify this
> for me? I am not a good programmer :(

   dynamic_cast<> can always return 0L.  You must be prepared to handle this 
case.  I'm sure in this case it *shouldn't* but that doesn't change the fact 
that it could, and if the code is changed in the future to add another 
incompatible object it could eventually legitimately fail.

-- 
George Staikos
KDE Developer				http://www.kde.org/
Staikos Computing Services Inc.		http://www.staikos.net/

[prev in list] [next in list] [prev in thread] [next in thread]