Re: [PATCH v1 2/2] t/*: fix ordering of expected/observed arguments

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Sep 14, 2018 at 5:03 PM Matthew DeVore <matvore@xxxxxxxxxx> wrote:
>
> This fixes various places where the ordering was obviously wrong and it
> was either related to other patches in this patchset or was easy
> find with grep.
>
> Signed-off-by: Matthew DeVore <matvore@xxxxxxxxxx>
> ---
The clause about "related to other patches in this patchset is wrong,
so I've reworded the commit message body:

    Fix various places where the ordering was obviously wrong, meaning it
    was easy to find with grep.

> --- a/t/t7508-status.sh
> +++ b/t/t7508-status.sh
> @@ -1620,7 +1620,7 @@ test_expect_success 'show stash info with "--show-stash"' '
>
>  test_expect_success 'no stash info with "--show-stash --no-show-stash"' '
>         git status --show-stash --no-show-stash >expected_without_stash &&
> -       test_cmp expected_default expected_without_stash
> +       test_cmp expected_without_stash expected_default
>  '
This line should not have been changed, so I've reverted it.



[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux