Use .at() method instead of operator[] for shareable worklet lookup (#604)

tomekzaw · web-flow · commit c3f616be2b55 · 2025-01-31T15:51:19.000+01:00
diff --git a/cpp/MarkdownGlobal.cpp b/cpp/MarkdownGlobal.cpp
@@ -36,9 +36,7 @@ void unregisterMarkdownWorklet(const int parserId) {
 
 std::shared_ptr<ShareableWorklet> getMarkdownWorklet(const int parserId) {
   std::unique_lock<std::mutex> lock(globalMarkdownShareableWorkletsMutex);
-  const auto &worklet = globalMarkdownShareableWorklets[parserId];
-  assert(worklet != nullptr);
-  return worklet;
+  return globalMarkdownShareableWorklets.at(parserId);
 }
 
 } // namespace livemarkdown

Original file line number	Diff line number	Diff line change
`@@ -36,9 +36,7 @@ void unregisterMarkdownWorklet(const int parserId) {`
`36`	`36`
`37`	`37`	`std::shared_ptr<ShareableWorklet> getMarkdownWorklet(const int parserId) {`
`38`	`38`	`std::unique_lock<std::mutex> lock(globalMarkdownShareableWorkletsMutex);`
`39`		`- const auto &worklet = globalMarkdownShareableWorklets[parserId];`
`40`		`- assert(worklet != nullptr);`
`41`		`- return worklet;`
	`39`	`+ return globalMarkdownShareableWorklets.at(parserId);`
`42`	`40`	`}`
`43`	`41`
`44`	`42`	`} // namespace livemarkdown`