comparison mercurial/error.py @ 46981:abd18d6306f1

errors: remove unnecessary varargs handling from OutOfBandError

In my recent D10465, I moved some code over from scmutil into
`OutOfBandError.__init__`. The code was written to deal with an
arbitrary number of `message` arguments to the constructor. It turns
out that we only ever pass 0 or 1. Given that, let's simplify it.

Differential Revision: https://phab.mercurial-scm.org/D10483

46980:75351b8b2082

class OutOfBandError(RemoteError):
    """Exception raised when a remote repo reports failure"""

    def __init__(self, *messages, **kwargs):
        from .i18n import _

        if messages:
            message = _(b"remote error:\n%s") % b''.join(messages)
            # Abort.format() adds a trailing newline
            message = message.rstrip(b'\n')
        else:
            message = _(b"remote error")
        super(OutOfBandError, self).__init__(message, **kwargs)


class ParseError(Abort):
    """Raised when parsing config files and {rev,file}sets (msg[, pos])"""

46981:abd18d6306f1

class OutOfBandError(RemoteError):
    """Exception raised when a remote repo reports failure"""

    def __init__(self, message=None, hint=None):
        from .i18n import _

        if message:
            # Abort.format() adds a trailing newline
            message = _(b"remote error:\n%s") % message.rstrip(b'\n')
        else:
            message = _(b"remote error")
        super(OutOfBandError, self).__init__(message, hint=hint)


class ParseError(Abort):
    """Raised when parsing config files and {rev,file}sets (msg[, pos])"""