From: Nemina Amarasinghe Date: Mon, 10 Mar 2014 08:51:05 GMT Subject: Re: [PATCH] simplified the chain if() statements of install_branch_config() function in branch.c Message-ID: In-Reply-To: > > Nemina Amarasinghe 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))