Re: [RFC PATCH] in-kernel lirc

[Jarod W. <ja...@wi...>]

On Wednesday 12 September 2007 03:21:31 pm Jarod Wilson wrote:
> On Sunday 09 September 2007 04:45:00 pm Christoph Bartelmus wrote:
> > I think the code that uses alloc_status, mem_faiure, etc. should be =C2=
=A0
> > rewritten to use goto or similar. At least the changes to avoid =C2=A0
> > assignments in if conditions were not vaild, so I had to undo some of =
=C2=A0
> > them.
>
> Does the attached patch snippet make sense as a route to go? If so, I'll
> apply the same treatment to other remaining similar sections.

Ugh, scratch that version... Instead of a goto inside every if, after all t=
he=20
ifs, test for alloc_status !=3D SUCCESS, and if so, then goto. Much cleaner.

=2D-=20
Jarod Wilson
ja...@wi...