upstream/mercurial-mirror Files · rust/rhg/src/commands/debugrequirements.rs

typing: add a fake `__init__()` to bytestr to distract pytype...

typing: add a fake `__init__()` to bytestr to distract pytype I'm not sure what changed before pytype 09-09-2021 (from 04-15-2021), but these started getting flagged. This wrapping an exception in a `bytestr` pattern has been flagged before, and I've fixed it then with `stringutil.forcebytestr()`. But that doesn't work here, because it would create a circular import. I suspect the issue is `bytes.__new__()` wants `Iterable[int]`, so it just assumes the subclass will also take that. The referenced pytype bug isn't an exact match, but seems related and the suggested workaround helps. The specific warnings fixed are: File "/mnt/c/Users/Matt/hg/mercurial/encoding.py", line 212, in tolocal: Function bytestr.__init__ was called with the wrong arguments [wrong-arg-types] Expected: (self, ints: Iterable[int]) Actually passed: (self, ints: LookupError) Attributes of protocol Iterable[int] are not implemented on LookupError: __iter__ Called from (traceback): line 353, in current file File "/mnt/c/Users/Matt/hg/mercurial/encoding.py", line 240, in fromlocal: Function bytestr.__init__ was called with the wrong arguments [wrong-arg-types] Expected: (self, ints: Iterable[int]) Actually passed: (self, ints: UnicodeDecodeError) Attributes of protocol Iterable[int] are not implemented on UnicodeDecodeError: __iter__ Differential Revision: https://phab.mercurial-scm.org/D11466

Simon Sapin - - Load All Authors

File last commit:

r47335:5ce2aa7c default


                r48819:1fda8c93

default

Download file

             debugrequirements.rs
        
                    22 lines
            
             | 625 B
            
                | application/rls-services+xml
            
             |
                RustLexer
            
             / rust / rhg / src / commands / debugrequirements.rs
          
                    History
                
                 |
                  Annotation
                 | Raw
                 |Copy content
                 |Copy permalink

      use crate::error::CommandError;

      pub const HELP_TEXT: &str = "

      Print the current repo requirements.

      ";

      pub fn args() -> clap::App<'static, 'static> {

          clap::SubCommand::with_name("debugrequirements").about(HELP_TEXT)

      }

      pub fn run(invocation: &crate::CliInvocation) -> Result<(), CommandError> {

          let repo = invocation.repo?;

          let mut output = String::new();

          let mut requirements: Vec<_> = repo.requirements().iter().collect();

          requirements.sort();

          for req in requirements {

              output.push_str(req);

              output.push('\n');

          }

          invocation.ui.write_stdout(output.as_bytes())?;

          Ok(())

      }

	Site-wide shortcuts
/	Use quick search box
g h	Goto home page
g g	Goto my private gists page
g G	Goto my public gists page
g 0-9	Goto bookmarked items from 0-9
n r	New repository page
n g	New gist page

	Repositories
g s	Goto summary page
g c	Goto changelog page
g f	Goto files page
g F	Goto files page with file search activated
g p	Goto pull requests page
g o	Goto repository settings
g O	Goto repository access permissions settings
t s	Toggle sidebar on some pages

				use crate::error::CommandError;

				pub const HELP_TEXT: &str = "
				Print the current repo requirements.
				";

				pub fn args() -> clap::App<'static, 'static> {
				clap::SubCommand::with_name("debugrequirements").about(HELP_TEXT)
				}

				pub fn run(invocation: &crate::CliInvocation) -> Result<(), CommandError> {
				let repo = invocation.repo?;
				let mut output = String::new();
				let mut requirements: Vec<_> = repo.requirements().iter().collect();
				requirements.sort();
				for req in requirements {
				output.push_str(req);
				output.push('\n');
				}
				invocation.ui.write_stdout(output.as_bytes())?;
				Ok(())
				}