Bjorn

> Hello Collada-Pluginfans,
> 
> here are some problems with collada plugin:
> 
> 1. Problem + Solution:
> -------------------------------------------------------------
> can someone please verify it this a feature or a bug in the current osg-
> svn
> version:
> 
> Assumed error in if statement:
> 
> in daeWriter line 92:
> ----- snip
> //### provide a name to node
> std::string daeWriter::getNodeName(const osg::Node & node,const
> std::string
> & defaultName)
> {
>     std::string nodeName;
>     if ((node.getName().empty()) || (node.getName()!="")) <-- assumed to
> be
> wrong
>         nodeName=uniquify(defaultName);
>     else
>         nodeName=node.getName();
>     return nodeName;
> }
> ---- snip
> 
> suggestion replace "!=" with "==":
>     if ((node.getName().empty()) || (node.getName()=="")) <-- assumed to
> be
> correct
>
[Roger James] Your fix looks correct to me. I suggest you post it to
osg-submission, but read the submissions protocol at
www.openscenegraph.org/projects/osg/wiki/MailingLists/SubmissionsProtocol
first.
 
> 2. Problem + Solution:
> ---------------------------------------------------------------
> Compiling current svn version of osg and current svn version of
> collada_dom
> 
> I got errors using vc7.1 and compiling daeRMaterials.cpp every time
> something like this occurred:
> NULL != SetParamArray[i]->getFx_basic_type_common()
> NULL != SetParamArray[i]->getFx_basic_type_common()->getFloat4()
> 
> Errormessage: no binary operator '!=': no global operator found accepting
> type 'const domFx_baisc_type_commonRef' (<-- this is a rough translation
> from German Visual-Studio to English ). It seems there is a problem in
> converting dae-smart-ptr I could eliminate errors by adding .cast() at the
> end:
> NULL != SetParamArray[i]->getFx_basic_type_common().cast()
> NULL != SetParamArray[i]->getFx_basic_type_common()->getFloat4().cast()
> 
> Did anyone else had the same Problem with vc7.1? Can this be integrated to
> current osg-svn?
> 
> 

[Roger James] This looks benign. It looks like VC7.1 does not like
overloaded cast operators or maybe it is a const issue.

Roger

_______________________________________________
osg-users mailing list
[email protected]
http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org

Reply via email to