<div dir="ltr">Hi Robbert, Björn,<div>as I am updating to visual studio 2015 too, I added the new FBX sdk for 2015, and get a single warning:</div><div><br></div><div><div>54>OpenSceneGraph\src\osgPlugins\fbx\WriterCompareTriangle.cpp(23): warning C4456: declaration of 'g' hides previous local declaration</div><div>54>  OpenSceneGraph\src\osgPlugins\fbx\WriterCompareTriangle.cpp(14): note: see declaration of 'g'</div></div><div><br></div><div>attached are a fix for the warning and an update for the FindFBX.cmake to find the new SDK for windows.</div><div>Regards, Laurens.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jun 6, 2016 at 3:41 PM, Robert Osfield <span dir="ltr"><<a href="mailto:robert.osfield@gmail.com" target="_blank">robert.osfield@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Björn,<br>
<span class=""><br>
On 6 June 2016 at 13:55, Björn Blissing <<a href="mailto:bjorn.blissing@vti.se">bjorn.blissing@vti.se</a>> wrote:<br>
> In the LUA-plugin:<br>
><br>
><br>
> Code:<br>
><br>
> d:\code\github\openscenegraph\src\osgplugins\lua\lua-5.2.3\src\lapi.c(1110): warning C4702: unreachable code<br>
<br>
</span>Removing the return statement that the warning is complaining about<br>
would force a in the break method API.  Could be a slippery slope of<br>
changes required so I've just disabled the warning, at least tried to<br>
using simple CMake entry to what you added to the 3ds plugin.<br>
<span class=""><br>
> In the OSC-plugin:<br>
><br>
><br>
> Code:<br>
> openscenegraph\src\osgPlugins\osc\ip\win32\NetworkingUtils.cpp(80): warning C4996: 'gethostbyname': Use getaddrinfo() or GetAddrInfoW() instead or define _WINSOCK_DEPRECATED_NO_WARNINGS to disable deprecated API warnings<br>
>   C:\Program Files (x86)\Windows Kits\8.1\Include\um\winsock2.h(2238): note: see declaration of 'gethostbyname'<br>
> openscenegraph\src\osgPlugins\osc\ip\win32\UdpSocket.cpp(401): warning C4456: declaration of 'currentTimeMs' hides previous local declaration<br>
>   openscenegraph\src\osgPlugins\osc\ip\win32\UdpSocket.cpp(386): note: see declaration of 'currentTimeMs'<br>
><br>
> The first one is self explainatory. The second one is a simple variable shadowing. Which I do not think have any unintended consequences.<br>
<br>
</span>This is 3rd party code that's integrated, we could fix or just disable<br>
the warnings.  If we modify the code we'd need to test it to make<br>
sure.  I'm not the original author of the code or a Windows hack so<br>
I'll not personally dive in and start hacking with the code.  If no<br>
one else feels the urge then I'd suggest we just disable the warning.<br>
<br>
The shadowing warning I've simply removed the second double as it's not needed.<br>
<span class=""><br>
<br>
> In the OSG-plugin:<br>
><br>
><br>
> Code:<br>
> openscenegraph\src\osgPlugins\osg\ReaderWriterOSG.cpp(257): warning C4459: declaration of 'NodeList' hides global declaration<br>
>   openscenegraph\include\osg/Group(22): note: see declaration of 'osg::NodeList'<br>
<br>
</span>I've changed the code to simply use the osg::NodeList variant.<br>
<span class=""><br>
<br>
> And finally in the TXP plugin:<br>
><br>
> Code:<br>
> openscenegraph\src\osgPlugins\txp\trpage_pparse.cpp(241): warning C4458: declaration of 'imageHelp' hides class member<br>
>   openscenegraph\src\osgPlugins\txp\trpage_print.h(136): note: see declaration of 'trpgPrintGraphParser::imageHelp'<br>
><br>
> This last one may actually have consequences which the author did not intend. But I am entirely not sure. The author have a protected member variable named imageHelp, which is set in the constructor. It also have an access method. But the variable is not used anywhere inside of the class, instead the scoped variable with the same name is used.<br>
<br>
</span>I've changed the local name to parse_imageHelp to make sure it's using<br>
the local one.<br>
<br>
With these fixes now checked into master it should just be the OSC warning left.<br>
<span class="HOEnZb"><font color="#888888"><br>
Robert.<br>
</font></span><div class="HOEnZb"><div class="h5">_______________________________________________<br>
osg-users mailing list<br>
<a href="mailto:osg-users@lists.openscenegraph.org">osg-users@lists.openscenegraph.org</a><br>
<a href="http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org" rel="noreferrer" target="_blank">http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org</a><br>
</div></div></blockquote></div><br></div>