comparison mercurial/localrepo.py @ 45197:55464c0b3a89

commitctx: move a tiny else clause above the very long one

This is simple to have all the simple case unfold before the 100+ line one. Otherwise it is hard to relate the `else` to the initial conditionnal.

This is part of a larger refactoring/cleanup of the commitctx code to clarify
and augment the logic gathering metadata useful for copy tracing. The current
code is a tad too long and entangled to make such update easy. We start with
easy and small cleanup.

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

45196:c068e355cd71

                mn = ctx.manifestnode()
                files = ctx.files()
                if writechangesetcopy:
                    filesadded = ctx.filesadded()
                    filesremoved = ctx.filesremoved()
            elif ctx.files():
                m1ctx = p1.manifestctx()
                m2ctx = p2.manifestctx()
                mctx = m1ctx.copy()

                m = mctx.read()

                    self.ui.debug(
                        b'reusing manifest from p1 (listed files '
                        b'actually unchanged)\n'
                    )
                    mn = p1.manifestnode()
            else:
                self.ui.debug(b'reusing manifest from p1 (no file change)\n')
                mn = p1.manifestnode()
                files = []

            if writecopiesto == b'changeset-only':
                # If writing only to changeset extras, use None to indicate that
                # no entry should be written. If writing to both, write an empty
                # entry to prevent the reader from falling back to reading

45197:55464c0b3a89

                mn = ctx.manifestnode()
                files = ctx.files()
                if writechangesetcopy:
                    filesadded = ctx.filesadded()
                    filesremoved = ctx.filesremoved()
            elif not ctx.files():
                self.ui.debug(b'reusing manifest from p1 (no file change)\n')
                mn = p1.manifestnode()
                files = []
            else:
                m1ctx = p1.manifestctx()
                m2ctx = p2.manifestctx()
                mctx = m1ctx.copy()

                m = mctx.read()

                    self.ui.debug(
                        b'reusing manifest from p1 (listed files '
                        b'actually unchanged)\n'
                    )
                    mn = p1.manifestnode()

            if writecopiesto == b'changeset-only':
                # If writing only to changeset extras, use None to indicate that
                # no entry should be written. If writing to both, write an empty
                # entry to prevent the reader from falling back to reading