Add default switch cases.

Bug 15387371

Change-Id: I97644bfd9dbdc1109630199bc897f8b2572d5e83
Reviewed-on: https://swiftshader-review.googlesource.com/4502
Reviewed-by: Alexis Hétu <sugoi@google.com>
Reviewed-by: Nicolas Capens <capn@google.com>
Tested-by: Nicolas Capens <capn@google.com>

diff --git a/src/Shader/PixelRoutine.cpp b/src/Shader/PixelRoutine.cpp
index c313eb4..c87f3cb 100644 (file)
--- a/src/Shader/PixelRoutine.cpp
+++ b/src/Shader/PixelRoutine.cpp
@@ -1400,6 +1400,8 @@ namespace sw
                                current.z = current.z - As<Short4>(As<UShort4>(current.z) >> 8) + Short4(0x0080, 0x0080, 0x0080, 0x0080);
                                current.w = current.w - As<Short4>(As<UShort4>(current.w) >> 8) + Short4(0x0080, 0x0080, 0x0080, 0x0080);
                                break;
+                       default:
+                               break;
                        }
                }
 

diff --git a/src/Shader/PixelShader.cpp b/src/Shader/PixelShader.cpp
index 0cd6f1c..2823aed 100644 (file)
--- a/src/Shader/PixelShader.cpp
+++ b/src/Shader/PixelShader.cpp
@@ -716,6 +716,8 @@ namespace sw
                                        case Shader::PARAMETER_TEXTURE:
                                                semantic[2 + reg][0].centroid = centroid;
                                                break;
+                                       default:
+                                               break;
                                        }
 
                                        this->centroid = this->centroid || centroid;

diff --git a/src/Shader/Shader.cpp b/src/Shader/Shader.cpp
index 82a038d..a0ece65 100644 (file)
--- a/src/Shader/Shader.cpp
+++ b/src/Shader/Shader.cpp
@@ -1578,6 +1578,8 @@ namespace sw
                                        dirtyConstantsB = instruction[i]->dst.index + 1;
                                }
                                break;
+                       default:
+                               break;
                        }
                }
        }
@@ -1627,6 +1629,8 @@ namespace sw
                        case OPCODE_DEFB:
                        case OPCODE_DEFI:
                                containsDefine = true;
+                       default:
+                               break;
                        }
                }
 
@@ -1798,6 +1802,8 @@ namespace sw
                                        }
                                }
                                break;
+                       default:
+                               break;
                        }
                }
        }
@@ -1837,6 +1843,7 @@ namespace sw
                                case PARAMETER_TEMP:   dynamicallyIndexedTemporaries = true; break;
                                case PARAMETER_INPUT:  dynamicallyIndexedInput = true;       break;
                                case PARAMETER_OUTPUT: dynamicallyIndexedOutput = true;      break;
+                               default: break;
                                }
                        }
 
@@ -1852,6 +1859,7 @@ namespace sw
                                        case PARAMETER_TEMP:   dynamicallyIndexedTemporaries = true; break;
                                        case PARAMETER_INPUT:  dynamicallyIndexedInput = true;       break;
                                        case PARAMETER_OUTPUT: dynamicallyIndexedOutput = true;      break;
+                                       default: break;
                                        }
                                }
                        }