parseDerivation(): Fix warning about uninitialized 'version' variable

NixOS · Nov 3, 2023 · 55dd124 · 55dd124
1 parent e9a857e
commit 55dd124
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/src/libstore/derivations.cc b/src/libstore/derivations.cc
@@ -352,7 +352,7 @@ Derivation parseDerivation(
  expect(str, "erive(");
  version = DerivationATermVersion::Traditional;
  break;
- case 'r':
+ case 'r': {
  expect(str, "rvWithVersion(");
  auto versionS = parseString(str);
  if (versionS == "xp-dyn-drv") {
@@ -365,6 +365,9 @@ Derivation parseDerivation(
  expect(str, ",");
  break;
  }
+ default:
+ throw Error("derivation does not start with 'Derive' or 'DrvWithVersion'");
+ }
 
  /* Parse the list of outputs. */
  expect(str, "[");