Re: Bug in "ide-pci.c"

• Next message: Tom Rini: "Re: RedHat 7.0 anaconda installer doesn't support devfs"
• Previous message: Forever shall I be.: "Re: [OOPS] unloading ns558 driver in Linux 2.4.0-test9"
• In reply to: Andre Hedrick: "Re: Bug in "ide-pci.c""
• Next in thread: Tom Leete: "Re: Bug in "ide-pci.c""
• Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

On Fri, 6 Oct 2000, Andre Hedrick wrote:

> void go_take_a_dump (float load)
> {
> if (pull_down_pants() && purge_bowles() && wipe_anus() && pull_up_pants())
> flush(load);
> else
> wear(load);
> }

But here you make another classic mistake. Consider the case where
purge_bowels() fails (-EWOULDBLOCK?).

In that case, you don't actually execute the subsequent two procedure
calls, which I'm sure we all agree is a suboptimal state of affairs.

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

• Next message: Tom Rini: "Re: RedHat 7.0 anaconda installer doesn't support devfs"
• Previous message: Forever shall I be.: "Re: [OOPS] unloading ns558 driver in Linux 2.4.0-test9"
• In reply to: Andre Hedrick: "Re: Bug in "ide-pci.c""
• Next in thread: Tom Leete: "Re: Bug in "ide-pci.c""
• Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

This archive was generated by hypermail 2b29 : Sat Oct 07 2000 - 21:00:20 EST