Cleanup TODOs

src/budget.cpp

@@ -270,8 +270,6 @@ int main(int argc, const char* argv[]) {
             code = 1;
         }
     } catch (const budget_exception& exception) {
-        // TODO We should be able to differentiate between real errors and
-        // command line errors
         std::cout << exception.message() << std::endl;
 
         code = 2;

src/config.cpp

@@ -352,7 +352,7 @@ bool budget::net_worth_over_fortune(){
     // By default, fortune is the thing being taken into account
     // Unless it's not used and net worth is used
 
-    // TODO This can be a very expensive operation!
+    // This can be a very expensive operation! If this ever causes issues, I should find a better solution
     return !no_asset_values() && no_fortunes();
 }
 

src/console_writer.cpp

@@ -336,5 +336,5 @@ void budget::console_writer::display_graph([[maybe_unused]] std::string_view
                                            [[maybe_unused]] std::vector<std::string>&        categories,
                                            [[maybe_unused]] std::vector<std::string>         series_names,
                                            [[maybe_unused]] std::vector<std::vector<float>>& series_values) {
-    os << "TODO: unimplemented" << std::endl;
+    os << "unimplemented" << std::endl;
 }

src/money.cpp

@@ -22,7 +22,7 @@ money budget::money_from_string(std::string_view money_sv){
 
     // In order to read locale-dependent data (legacy), we need
     // to allow , in the numbers
-    // TODO In the future, we can remove this code
+    // In the future, we can remove this code
     std::erase(money_string, ',');
 
     int dollars = 0;