fix: pass std::nullopt instead of NULL pointer to BatchQuery

chrispader · chrispader · commit fa86b70e40a8 · 2025-03-03T16:58:16.000+01:00
diff --git a/package/cpp/sqliteExecuteBatch.cpp b/package/cpp/sqliteExecuteBatch.cpp
@@ -25,7 +25,7 @@ std::vector<BatchQuery> batchParamsToCommands(const std::vector<BatchQueryComman
         commands.push_back(BatchQuery{command.query, std::move(std::get<ParamsVec>(*command.params))});
       }
     } else {
-      commands.push_back(BatchQuery{command.query, NULL});
+      commands.push_back(BatchQuery{command.query, std::nullopt});
     }
   }
 

Original file line number	Diff line number	Diff line change
`@@ -25,7 +25,7 @@ std::vector<BatchQuery> batchParamsToCommands(const std::vector<BatchQueryComman`
`25`	`25`	`commands.push_back(BatchQuery{command.query, std::move(std::get<ParamsVec>(*command.params))});`
`26`	`26`	`}`
`27`	`27`	`} else {`
`28`		`- commands.push_back(BatchQuery{command.query, NULL});`
	`28`	`+ commands.push_back(BatchQuery{command.query, std::nullopt});`
`29`	`29`	`}`
`30`	`30`	`}`
`31`	`31`