doesnt need to be explicit

WisoAltred · WisoAltred · commit 98f2e4dc4518 · 2026-02-02T09:34:53.000+02:00
diff --git a/src/courtroom.cpp b/src/courtroom.cpp
@@ -3525,9 +3525,8 @@ static std::optional<PauseInfo> parse_pause_duration(const QString &text, int st
     return PauseInfo{1000, 0};
   }
 
-  bool ok;
-  int value = qMin(10000, text.mid(start_pos, pos - start_pos).toInt(&ok));
-  return ok ? std::optional<PauseInfo>{PauseInfo{value, pos - start_pos}} : std::nullopt;
+  int value = qMin(10000, text.mid(start_pos, pos - start_pos).toInt());
+  return std::optional<PauseInfo>{PauseInfo{value, pos - start_pos}};
 }
 
 QString Courtroom::filter_ic_text(QString p_text, bool html, int target_pos, int default_color)

Original file line number	Diff line number	Diff line change
`@@ -3525,9 +3525,8 @@ static std::optional<PauseInfo> parse_pause_duration(const QString &text, int st`
`3525`	`3525`	`return PauseInfo{1000, 0};`
`3526`	`3526`	`}`
`3527`	`3527`
`3528`		`- bool ok;`
`3529`		`- int value = qMin(10000, text.mid(start_pos, pos - start_pos).toInt(&ok));`
`3530`		`- return ok ? std::optional<PauseInfo>{PauseInfo{value, pos - start_pos}} : std::nullopt;`
	`3528`	`+ int value = qMin(10000, text.mid(start_pos, pos - start_pos).toInt());`
	`3529`	`+ return std::optional<PauseInfo>{PauseInfo{value, pos - start_pos}};`
`3531`	`3530`	`}`
`3532`	`3531`
`3533`	`3532`	`QString Courtroom::filter_ic_text(QString p_text, bool html, int target_pos, int default_color)`