few compile warnings - Githubissues

mosra / magnum

Lightweight and modular C++11 graphics middleware for games and data visualization

https://magnum.graphics/

Other

4.74k stars 439 forks source link

few compile warnings #611

Closed linuxaged closed 1 year ago

linuxaged commented 1 year ago

magnum\src\Magnum\ImageFlags.cpp(43): warning C4060: switch statement contains no 'case' or 'default' labels
magnum\src\Magnum\GL\Context.cpp(1012): warning C4456: declaration of 'state' hides previous local declaration
magnum\src\Magnum\Trade\MaterialData.cpp(352): warning C4456: declaration of 'i' hides previous local declaration
magnum\src\Magnum\Trade\AbstractSceneConverter.cpp(1315): warning C4456: declaration of 'j' hides previous local declaration

mosra commented 1 year ago

Thank you! Should be fixed in 2e3baf1911295ffde78edf09c594ca04697b981f and 28c1f84e689e1380867c16d3dce38df406316ae3 -- I found a few more variable shadowing warnings and fixed them as well.

Oh, and sorry for the other warnings, reported in #544 -- I promise I'll fix them eventually as well :)