[Paraview] pqLookmarkStateLoader.cxx bug?

Utkarsh Ayachit utkarsh.ayachit at kitware.com
Thu Mar 27 08:54:30 EDT 2008


Thanks for noting this issue. I've committed a fix to CVS. Here's the patch.

Index: Qt/Components/pqLookmarkStateLoader.cxx
===================================================================
RCS file: 
/cvsroot/ParaView3/ParaView3/Qt/Components/pqLookmarkStateLoader.cxx,v
retrieving revision 1.19
diff -u -3 -p -r1.19 pqLookmarkStateLoader.cxx
--- Qt/Components/pqLookmarkStateLoader.cxx     5 Dec 2007 14:56:41 
-0000      1.19
+++ Qt/Components/pqLookmarkStateLoader.cxx     27 Mar 2008 12:50:46 -0000
@@ -444,7 +444,7 @@ void pqLookmarkStateLoader::RegisterProx
      return;
      }

-  return this->Superclass::RegisterProxy(id, proxy);
+  this->Superclass::RegisterProxy(id, proxy);
  }


Utkarsh

Frederick Isaac wrote:
> code is as follows
> 
> void pqLookmarkStateLoader::RegisterProxy(int id,
> vtkSMProxy* proxy)
> {
>   // Don't register a proxy that we are going to
> ignore later
>  
> if(this->Internal->IdsOfProxyElementsToIgnore.contains(QString::number(id)))
>     {
>     return;
>     }
> 
>   return this->Superclass::RegisterProxy(id, proxy);
> }
> 
> 
> ERROR: return returns value in void function
> 
> What should this be? any ideas?
> 
> Cheers
> 
> 
> 
> 
>       ____________________________________________________________________________________
> Looking for last minute shopping deals?  
> Find them fast with Yahoo! Search.  http://tools.search.yahoo.com/newsearch/category.php?category=shopping
> _______________________________________________
> ParaView mailing list
> ParaView at paraview.org
> http://www.paraview.org/mailman/listinfo/paraview
> 


More information about the ParaView mailing list