On Wed, Jan 20, 2010 at 07:30:47PM +0100, Jim Meyering wrote:
This avoids a leak and the need for a "cleanup:" block,
along with its three goto statements.
While often I prefer to write functions with a single return point,
this one no longer has the need, now that "addr" is freed immediately
after allocation.
Adding the semicolon in the "case..." stmt may look odd.
It's there because the first stmt is the declaration of "port".
If you'd prefer, an alternative is to put the contents of that
case inside a {...} block.
yeah, I really think a {} block is nicer in that case,
ACK,
Daniel
--
Daniel Veillard | libxml Gnome XML XSLT toolkit
http://xmlsoft.org/
daniel(a)veillard.com | Rpmfind RPM search engine
http://rpmfind.net/
http://veillard.com/ | virtualization library
http://libvirt.org/