Don't use dynamic cast if they are useless.

And when they are, please, check the return value...

Don't use dynamic cast if they are useless.
And when they are, please, check the return value...
e5579874 · Hugo Beauzee-Luyssen · b3a50081 · e5579874
Commit e5579874 authored Feb 19, 2010 by Hugo Beauzee-Luyssen
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 1 deletion

src/Library/Library.cpp src/Library/Library.cpp +1 -1

No files found.
--- a/src/Library/Library.cpp
+++ b/src/Library/Library.cpp
@@ -68,7 +68,7 @@ Library::clip( const QUuid& uuid )
    {
        Media* media = m_medias.value( mediaUuid );
        if ( media != NULL && media->clips()->contains( uuid ) )
-            return dynamic_cast<Clip*>( getElementByUuid( *media->clips(), uuid ) );
+            return getElementByUuid( *media->clips(), uuid );
    }
    return NULL;
 }