Fix warning 'catching polymorphic type by value'
This commit is contained in:
parent
3420b60b39
commit
5c95b32c46
|
@ -96,7 +96,7 @@ void CreateDerivationAndRealiseGoal::getDerivation()
|
||||||
auto drvPath = StorePath::dummy;
|
auto drvPath = StorePath::dummy;
|
||||||
try {
|
try {
|
||||||
drvPath = resolveDerivedPath(worker.store, *drvReq);
|
drvPath = resolveDerivedPath(worker.store, *drvReq);
|
||||||
} catch (MissingRealisation) {
|
} catch (MissingRealisation &) {
|
||||||
return std::nullopt;
|
return std::nullopt;
|
||||||
}
|
}
|
||||||
return worker.evalStore.isValidPath(drvPath) || worker.store.isValidPath(drvPath)
|
return worker.evalStore.isValidPath(drvPath) || worker.store.isValidPath(drvPath)
|
||||||
|
|
|
@ -60,7 +60,7 @@ void initLibUtil() {
|
||||||
bool caught = false;
|
bool caught = false;
|
||||||
try {
|
try {
|
||||||
throwExceptionSelfCheck();
|
throwExceptionSelfCheck();
|
||||||
} catch (nix::Error _e) {
|
} catch (const nix::Error & _e) {
|
||||||
caught = true;
|
caught = true;
|
||||||
}
|
}
|
||||||
// This is not actually the main point of this check, but let's make sure anyway:
|
// This is not actually the main point of this check, but let's make sure anyway:
|
||||||
|
|
Loading…
Reference in a new issue