Consolidate more error messages inside mathics.builtin.messages (#1838)

That is in the General section. This will be needed for PyICU improvements.

Author: rocky

mathics/autoload/formats/Image/Import.m

@@ -12,7 +12,7 @@
     FunctionChannels -> {"FileNames"}
 ];
 
-(* FIXME: RegisterImageImport shoudl work with MIME types, not psuedo-canonicalized file extensions. *)
+(* FIXME: RegisterImageImport should work with MIME types, not psuedo-canonicalized file extensions. *)
 RegisterImageImport[#]& /@ {"BMP", "GIF", "JPEG2000", "JPEG", "JPG", "PCX", "PNG", "PPM", "PBM", "PGM", "TIFF", "ICO", "TGA"};
 
 End[]

mathics/builtin/files_io/importexport.py

@@ -22,7 +22,6 @@
 import sys
 import urllib.request as request
 from itertools import chain
-from typing import Dict, Final
 from urllib.error import HTTPError, URLError
 
 from mathics.core.atoms import ByteArray

mathics/builtin/forms/data.py

@@ -128,7 +128,6 @@ class _NumberForm(Builtin):
             "Value for option NumberSeparator -> `1` should be a string "
             "or a pair of strings."
         ),
-        "opttf": "Value of option `1` -> `2` should be True or False.",
         "sigz": (
             "In addition to the number of digits requested, one or more "
             "zeros will appear as placeholders."

mathics/builtin/list/predicates.py

@@ -40,7 +40,6 @@ class ContainsOnly(Builtin):
 
     messages = {
         "lsa": "List or association expected instead of `1`.",
-        "nodef": "Unknown option `1` for ContainsOnly.",
         "optx": "Unknown option `1` in `2`.",
     }
 

mathics/builtin/messages.py

@@ -220,6 +220,7 @@ class General(Builtin):
         "matsq": "Argument `1` is not a non-empty square matrix.",
         "newpkg": "In WL, there is a new package for this.",
         "nffil": "File not found during `1`.",
+        "nodef": "Unknown option `1` for `2`.",
         "nonopt": "Options expected (instead of `1`) beyond position `2` in `3`. An option must be a rule or a list of rules.",
         "noopen": "Cannot open `1`.",
         "nord": "Invalid comparison with `1` attempted.",
@@ -228,6 +229,7 @@ class General(Builtin):
         "obspkg": "In WL, this package is obsolete.",
         "openx": "`1` is not open.",
         "optb": "Optional object `1` in `2` is not a single blank.",
+        "opttf": "Value of option `1` -> `2` should be True or False.",
         "ovfl": "Overflow occurred in computation.",
         "partd": "Part specification is longer than depth of object.",
         "partw": "Part `1` of `2` does not exist.",

mathics/builtin/numbers/algebra.py

@@ -75,7 +75,6 @@ class _Algebraic(Builtin):
 
     messages = {
         "modn": "Value of option `1` -> `2` should be an integer.",
-        "opttf": "Value of option `1` -> `2` should be True or False.",
         "modulus": "Modulus option not supported yet.",
     }
 
@@ -642,7 +641,6 @@ class _Expand(Builtin):
 
     messages = {
         "modn": "Value of option `1` -> `2` should be an integer.",
-        "opttf": "Value of option `1` -> `2` should be True or False.",
     }
 
     def convert_options(self, options: dict, evaluation: Evaluation):