simplify HTTP4xxClientError subclasses

Author: seeM

fastcore/_modidx.py

@@ -333,7 +333,6 @@
             'fastcore.net': { 'fastcore.net.HTTP4xxClientError': ('net.html#http4xxclienterror', 'fastcore/net.py'),
                               'fastcore.net.HTTP5xxServerError': ('net.html#http5xxservererror', 'fastcore/net.py'),
                               'fastcore.net.Request.summary': ('net.html#request.summary', 'fastcore/net.py'),
-                              'fastcore.net._mk_error': ('net.html#_mk_error', 'fastcore/net.py'),
                               'fastcore.net._socket_det': ('net.html#_socket_det', 'fastcore/net.py'),
                               'fastcore.net.do_request': ('net.html#do_request', 'fastcore/net.py'),
                               'fastcore.net.start_client': ('net.html#start_client', 'fastcore/net.py'),

fastcore/net.py

@@ -89,15 +89,11 @@ def urlopener():
     (431,'Header Fields Too Large'),(451,'Legal Reasons')
 )
 
-def _mk_error(nm, code, msg):
-    cls = get_class(nm, 'url', 'hdrs', 'fp', sup=HTTP4xxClientError, msg=msg, code=code)
-    def _init(self, url, hdrs, fp, msg=msg, code=code): super(cls, self).__init__(url, code, msg, hdrs, fp)
-    cls.__init__ = _init
-    return cls
-
 for code,msg in _httperrors:
     nm = f'HTTP{code}{msg.replace(" ","")}Error'
-    globals()[nm] = ExceptionsHTTP[code] = _mk_error(nm, code, msg)
+    def _init(self, url, hdrs, fp, msg=msg, code=code): HTTP4xxClientError.__init__(self, url, code, msg, hdrs, fp)
+    cls = type(nm, (HTTP4xxClientError,), {'__init__':_init})
+    globals()[nm] = ExceptionsHTTP[code] = cls
 
 # %% ../nbs/03b_net.ipynb 16
 _all_ = ['HTTP400BadRequestError', 'HTTP401UnauthorizedError', 'HTTP402PaymentRequiredError', 'HTTP403ForbiddenError', 'HTTP404NotFoundError', 'HTTP405MethodNotAllowedError', 'HTTP406NotAcceptableError', 'HTTP407ProxyAuthRequiredError', 'HTTP408RequestTimeoutError', 'HTTP409ConflictError', 'HTTP410GoneError', 'HTTP411LengthRequiredError', 'HTTP412PreconditionFailedError', 'HTTP413PayloadTooLargeError', 'HTTP414URITooLongError', 'HTTP415UnsupportedMediaTypeError', 'HTTP416RangeNotSatisfiableError', 'HTTP417ExpectationFailedError', 'HTTP418AmAteapotError', 'HTTP421MisdirectedRequestError', 'HTTP422UnprocessableEntityError', 'HTTP423LockedError', 'HTTP424FailedDependencyError', 'HTTP425TooEarlyError', 'HTTP426UpgradeRequiredError', 'HTTP428PreconditionRequiredError', 'HTTP429TooManyRequestsError', 'HTTP431HeaderFieldsTooLargeError', 'HTTP451LegalReasonsError']

nbs/03b_net.ipynb

@@ -184,7 +184,7 @@
       "text/markdown": [
        "---\n",
        "\n",
-       "[source](https://github.com/fastai/fastcore/blob/master/fastcore/net.py#L63){target=\"_blank\" style=\"float:right; font-size:smaller\"}\n",
+       "[source](https://github.com/fastai/fastcore/blob/master/fastcore/net.py#L64){target=\"_blank\" style=\"float:right; font-size:smaller\"}\n",
        "\n",
        "#### HTTP4xxClientError\n",
        "\n",
@@ -195,7 +195,7 @@
       "text/plain": [
        "---\n",
        "\n",
-       "[source](https://github.com/fastai/fastcore/blob/master/fastcore/net.py#L63){target=\"_blank\" style=\"float:right; font-size:smaller\"}\n",
+       "[source](https://github.com/fastai/fastcore/blob/master/fastcore/net.py#L64){target=\"_blank\" style=\"float:right; font-size:smaller\"}\n",
        "\n",
        "#### HTTP4xxClientError\n",
        "\n",
@@ -223,7 +223,7 @@
       "text/markdown": [
        "---\n",
        "\n",
-       "[source](https://github.com/fastai/fastcore/blob/master/fastcore/net.py#L68){target=\"_blank\" style=\"float:right; font-size:smaller\"}\n",
+       "[source](https://github.com/fastai/fastcore/blob/master/fastcore/net.py#L69){target=\"_blank\" style=\"float:right; font-size:smaller\"}\n",
        "\n",
        "#### HTTP5xxServerError\n",
        "\n",
@@ -234,7 +234,7 @@
       "text/plain": [
        "---\n",
        "\n",
-       "[source](https://github.com/fastai/fastcore/blob/master/fastcore/net.py#L68){target=\"_blank\" style=\"float:right; font-size:smaller\"}\n",
+       "[source](https://github.com/fastai/fastcore/blob/master/fastcore/net.py#L69){target=\"_blank\" style=\"float:right; font-size:smaller\"}\n",
        "\n",
        "#### HTTP5xxServerError\n",
        "\n",
@@ -284,15 +284,11 @@
     "    (431,'Header Fields Too Large'),(451,'Legal Reasons')\n",
     ")\n",
     "\n",
-    "def _mk_error(nm, code, msg):\n",
-    "    cls = get_class(nm, 'url', 'hdrs', 'fp', sup=HTTP4xxClientError, msg=msg, code=code)\n",
-    "    def _init(self, url, hdrs, fp, msg=msg, code=code): super(cls, self).__init__(url, code, msg, hdrs, fp)\n",
-    "    cls.__init__ = _init\n",
-    "    return cls\n",
-    "\n",
     "for code,msg in _httperrors:\n",
     "    nm = f'HTTP{code}{msg.replace(\" \",\"\")}Error'\n",
-    "    globals()[nm] = ExceptionsHTTP[code] = _mk_error(nm, code, msg)"
+    "    def _init(self, url, hdrs, fp, msg=msg, code=code): HTTP4xxClientError.__init__(self, url, code, msg, hdrs, fp)\n",
+    "    cls = type(nm, (HTTP4xxClientError,), {'__init__':_init})\n",
+    "    globals()[nm] = ExceptionsHTTP[code] = cls"
    ]
   },
   {