Messages in this thread | | | Date | Sat, 20 Dec 2014 16:48:10 +0100 | From | Lino Sanfilippo <> | Subject | Re: [PATCH v2 0/6] net-PPP: Deletion of a few unnecessary checks |
| |
Hi Markus,
On 20.12.2014 15:45, SF Markus Elfring wrote: >> I'm saying to leave the code alone. > > Do I need to try another interpretation out for your feedback? > > >> If it goes: >> >> var = foo_that_returns_ptr_err() >> if (IS_ERR(var)) >> return PTR_ERR(var); >> >> p->bar = var; >> >> or whatever, simply keep it that way! > > Do you want to express here that a data structure member should > only be set after a previous function call succeeded? >
I think what David said was pretty clear: If you see code like the above there is no need to refactor it. That does not mean that this is the _preferred_ way of error handling. Its just good enough to be left alone.
Regards, Lino
| |