Replace convertTo/stringify with non-throwing alternatives.

Having these throw is terrible for ergonomics and there are loads
of places where the exception was either silently ignored or not
handled at all. Having a function which returns an optional and
another that returns a default works a lot better imo.

1 files changed, 2 insertions, 2 deletions

diff --git a/modules/operserv/os_list.cpp b/modules/operserv/os_list.cpp
index a6969dc24..ba1156967 100644
--- a/modules/operserv/os_list.cpp
+++ b/modules/operserv/os_list.cpp
@@ -60,7 +60,7 @@ public:
 
 				ListFormatter::ListEntry entry;
 				entry["Name"] = cc->chan->name;
-				entry["Users"] = stringify(cc->chan->users.size());
+				entry["Users"] = Anope::ToString(cc->chan->users.size());
 				entry["Modes"] = cc->chan->GetModes(true, true);
 				entry["Topic"] = cc->chan->topic;
 				list.AddEntry(entry);
@@ -87,7 +87,7 @@ public:
 
 				ListFormatter::ListEntry entry;
 				entry["Name"] = c->name;
-				entry["Users"] = stringify(c->users.size());
+				entry["Users"] = Anope::ToString(c->users.size());
 				entry["Modes"] = c->GetModes(true, true);
 				entry["Topic"] = c->topic;
 				list.AddEntry(entry);