Re: [PATCH] x86: DS cleanup - dont treat 0 as NULL

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

From: H. Peter Anvin

Subject: Re: [PATCH] x86: DS cleanup - dont treat 0 as NULL

Date: Monday, April 28, 2008 - 11:38 pm

David Newall wrote:
quoted text> Cyrill Gorcunov wrote:
>> -			return 0;
>> +			return NULL;
> 
> Why?  The constant 0 is C's official symbol for a null pointer.

It's still bad style.

	-hpa
--
unsubscribe noticeTo unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:

[PATCH] x86: DS cleanup - dont treat 0 as NULL, Cyrill Gorcunov, (Mon Apr 28, 12:15 pm)

Re: [PATCH] x86: DS cleanup - dont treat 0 as NULL, H. Peter Anvin, (Mon Apr 28, 12:45 pm)

Re: [PATCH] x86: DS cleanup - dont treat 0 as NULL, H. Peter Anvin, (Mon Apr 28, 12:46 pm)

Re: [PATCH] x86: DS cleanup - dont treat 0 as NULL, Cyrill Gorcunov, (Mon Apr 28, 1:00 pm)

Re: [PATCH] x86: DS cleanup - dont treat 0 as NULL, Ingo Molnar, (Mon Apr 28, 1:02 pm)

Re: [PATCH] x86: DS cleanup - dont treat 0 as NULL, David Newall, (Mon Apr 28, 11:23 pm)

Re: [PATCH] x86: DS cleanup - dont treat 0 as NULL, H. Peter Anvin, (Mon Apr 28, 11:38 pm)

Re: [PATCH] x86: DS cleanup - dont treat 0 as NULL, Cyrill Gorcunov, (Tue Apr 29, 12:50 am)

Re: [PATCH] x86: DS cleanup - dont treat 0 as NULL, David Newall, (Tue Apr 29, 10:30 am)


linux-kernel:
Ken Chen	[patch] sched: fix inconsistency when redistribute per-cpu tg->cfs_rq shares.
Ingo Molnar	Re: [PATCH v3] x86: merge the simple bitops and move them to bitops.h
Jan Engelhardt	Re: [PATCH] Allow Kconfig to set default mmap_min_addr protection
Dmitry Torokhov	Re: [2.6 patch] input/serio/hp_sdc.c section fix
Rafael J. Wysocki	[Bug #16380] Loop devices act strangely in 2.6.35
git:
Steven Grimm	Using git as a general backup mechanism (was Re: Using GIT to store /etc)
Jeff King	Re: [PATCH] git-reset: allow --soft in a bare repo
Johannes Sixt	Re: [PATCH 01/14] msvc: Fix compilation errors in compat/win32/sys/poll.c
Johannes Schindelin	Re: [PATCH] Uninstall rule for top level Makefile
Shawn O. Pearce	Re: [PATCH v2] Speed up bash completion loading
git-commits-head:
Linux Kernel Mailing List	cgroups: clean up cgroup_pidlist_find() a bit
Linux Kernel Mailing List	sony-laptop: Add support for extended hotkeys
Linux Kernel Mailing List	IB/core: Add support for masked atomic operations
Linux Kernel Mailing List	V4L/DVB (8939): cx18: fix sparse warnings
Linux Kernel Mailing List	ipv6 mcast: Check address family of gf_group in getsockopt(MS_FILTER).
linux-netdev:
Inaky Perez-Gonzalez	[PATCH 40/40] wimax/i2400m: add CREDITS and MAINTAINERS entries
Karsten Keil	[mISDN PATCH v2 05/19] Reduce stack size in dsp_cmx_send()
linux	Re: 2.6.23-rc8 network problem. Mem leak? ip1000a?
David Miller	Re: tun: Use netif_receive_skb instead of netif_rx
David Miller	Re: [net-next PATCH v2] llc enhancements
freebsd-current:
Matthew Fleming	Re: [RFC] Outline of USB process integration in the kernel taskqueue system
illoai@gmail.com	Re: OT: 2d password
Hartmut Brandt	Re: problem with nss_ldap
Andrew Reilly	Re: FreeBSD's problems as seen by the BSDForen.de community
Max Laier	Re: Upcoming ABI Breakage in RELENG_7