git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: Alternative approach to the git config NULL value checking patches..

From
Linus Torvalds <torvalds@linux-foundation.org>
Date
Feb 10, 2008, 23:36 UTC
Message-ID
<alpine.LFD.1.00.0802101532070.2920@woody.linux-foundation.org>
In-Reply-To
<7vbq6oe98y.fsf@gitster.siamese.dyndns.org>
On Sun, 10 Feb 2008, Junio C Hamano wrote:
Show 5 quoted lines
> >
> > And this is different from checking against NULL exactly how?
> 
> Exactly.  My answer to your question is: "It is not different
> from checking against NULL at all."
Of course it is.
Not using NULL means that things like
	strcmp()
	atoi()
	..
all work and automatically get the right answer and don't need to care.
Take a look at the NULL-compare patches. 90% of them are just noise.

Yes, the 10% that actually *cares* will look exactly the same (the difference being comparing against NULL or config_true), but all the stuff that compares against a special string can just ignore the fact that the special truth value will never ever match that string.

		Linus
Previous: Linus TorvaldsNext: Linus Torvalds
Message 8 of 13 in “Alternative approach to the git config NULL value checking patches..”
  1. Linus TorvaldsFeb 10, 2008
  2. Junio C HamanoFeb 10, 2008
  3. Linus TorvaldsFeb 10, 2008
  4. Junio C HamanoFeb 10, 2008
  5. Junio C HamanoFeb 10, 2008
  6. Pierre HabouzitFeb 10, 2008
  7. Linus TorvaldsFeb 10, 2008
  8. Linus TorvaldsFeb 10, 2008
  9. Linus TorvaldsFeb 10, 2008
  10. Junio C HamanoFeb 11, 2008
  11. Christian CouderFeb 11, 2008
  12. Junio C HamanoFeb 11, 2008
  13. Daniel BarkalowFeb 11, 2008

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.