D10235: typing: mark the argument to mercurial.i18n.gettext() non-Optional

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

D10235: typing: mark the argument to mercurial.i18n.gettext() non-Optional

marmoute (Pierre-Yves David)
mharbison72 created this revision.
Herald added a reviewer: hg-reviewers.
Herald added a subscriber: mercurial-patches.

REVISION SUMMARY
  Few if any of the callers are handling a `None` return, which is making pytype
  complain.  I tried adding @overload definitions to indicate the bytes -> bytes
  and None -> None relationship, but pytype doesn't seem to apply that to `_()`
  through the function assignment.  What did work was to change `_()` into its own
  function that called `gettext()`, but that has an extra function call overhead.
  Even putting that function into an `if pycompat.TYPE_CHECKING` block and leaving
  the existing assignments in the `else` block caused pytype to lose track of the
  @overloads.
 
  At that point, I simply gave up.  PyCharm doesn't like that it can return None,
  given the new type hints, but pytype doesn't complain about this nor does it
  see any callers passing None.  The most important thing here is to catch str
  being passed anyway.

REPOSITORY
  rHG Mercurial

BRANCH
  default

REVISION DETAIL
  https://phab.mercurial-scm.org/D10235

AFFECTED FILES
  mercurial/i18n.py

CHANGE DETAILS

diff --git a/mercurial/i18n.py b/mercurial/i18n.py
--- a/mercurial/i18n.py
+++ b/mercurial/i18n.py
@@ -23,7 +23,6 @@
     from typing import (
         Callable,
         List,
-        Optional,
     )
 
 
@@ -71,7 +70,7 @@
 
 
 def gettext(message):
-    # type: (Optional[bytes]) -> Optional[bytes]
+    # type: (bytes) -> bytes
     """Translate message.
 
     The message is looked up in the catalog to get a Unicode string,



To: mharbison72, #hg-reviewers
Cc: mercurial-patches, mercurial-devel
_______________________________________________
Mercurial-devel mailing list
[hidden email]
https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel