Merge pull request #18 from srk-lunarg/v30_static_analysis_fixes

Comment out some dead assignments per suggestion from static analysis…

Merge pull request #18 from srk-lunarg/v30_static_analysis_fixes
ace4c45a · John Kessenich · a417f010 · 465a1466 · ace4c45a
Commit ace4c45a authored Jul 15, 2015 by John Kessenich
Hide whitespace changes
Inline Side-by-side

Showing with 7 additions and 6 deletions

SPVRemapper.cpp SPIRV/SPVRemapper.cpp +7 -6

No files found.
--- a/SPIRV/SPVRemapper.cpp
+++ b/SPIRV/SPVRemapper.cpp
@@ -450,11 +450,12 @@ namespace spv {
                return nextInst;

            case spv::OperandVariableLiterals:
-                if (opCode == spv::OpDecorate && asDecoration(word - 1) == spv::DecorationBuiltIn) {
-                    ++word;
-                    --numOperands;
-                }
-                word += numOperands;
+                // for clarity
+                // if (opCode == spv::OpDecorate && asDecoration(word - 1) == spv::DecorationBuiltIn) {
+                //     ++word;
+                //     --numOperands;
+                // }
+                // word += numOperands;
                return nextInst;

            case spv::OperandVariableLiteralId:
@@ -466,7 +467,7 @@ namespace spv {
                return nextInst;

            case spv::OperandLiteralString:
-                word += literalStringWords(literalString(word));
+                // word += literalStringWords(literalString(word)); // for clarity
                return nextInst;

                // Single word operands we simply ignore, as they hold no IDs