Re: [PATCH] simplified the chain if() statements of install_branch_config() function in branch.c
- From
David Kastrup <dak@gnu.org>
- Date
- Mar 10, 2014, 09:08 UTC
- Message-ID
- <878usia09y.fsf@fencepost.gnu.org>
- In-Reply-To
- <loom.20140310T094407-5@post.gmane.org>
Nemina Amarasinghe <neminaa@gmail.com> writes:
Show 9 quoted lines
>> >> Nemina Amarasinghe <neminaa <at> gmail.com> writes: >> >> Is it me, or is (origin || !origin) a tautology? >> > Thanks for the advices Matthieu. I will go through the documentations again. > Is there anything wrong with my logic? > What I wanted to express is > ((!remote_is_branch && origin) || (!remote_is_branch || !origin))
Is it?
The above is the same as (!remote_is_branch || !origin). What you wrote before is the same as (!remote_is_branch).
Maybe you should try copy&paste from the expressions you are trying to combine to make sure that what you start with makes sense.
-- David Kastrup