Re: BUG_ON()

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

 



Hi,

"Stoyan Gaydarov" <stoyboyker@xxxxxxxxx> writes:

> I was wondering if someone could clear something up for me.
> Is this desired or is it an unneeded change:
>
> from:
>
> if(someCondition)
>       BUG();
>
> to:
>
> BUG_ON(sameCondition);
>
> if it is desired then would anyone care if I took a crack at it?

I don't think it is worth the effort.  It's not that the first usage is
deprecated, I heard people insisting on the first way when it was
critisized in a review because they would find it more readable.

Both versions are obvious and it probably creates more trouble than it
is worth it.

Why not try something you learn from? :)

	Hannes

--
To unsubscribe from this list: send an email with
"unsubscribe kernelnewbies" to ecartis@xxxxxxxxxxxx
Please read the FAQ at http://kernelnewbies.org/FAQ


[Index of Archives]     [Newbies FAQ]     [Linux Kernel Mentors]     [Linux Kernel Development]     [IETF Annouce]     [Git]     [Networking]     [Security]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux SCSI]     [Linux ACPI]
  Powered by Linux