Re: [PATCH 5/13: eCryptfs] Header declarations

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

 



Pekka Enberg wrote:
Pekka Enberg wrote:

So, what's wrong with BUG_ON?

I guess because this may be compiled off when no debugging/extra checking is needed.
But you shouldn't write assertions that you don't really need anyway.

I wouldn't be so strict. Normally, you don't need them and disable as they slow you down. But when you are hunting a problem you enable them as they may catch it on an early stage. I don't see anything bad here...

--
Best regards, Artem B. Bityutskiy
Oktet Labs (St. Petersburg), Software Engineer.
+7 812 4286709 (office) +7 911 2449030 (mobile)
E-mail: [email protected], Web: www.oktetlabs.ru
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux