[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH 1/1] move exit from module_close to main
From: |
William Hubbs |
Subject: |
[PATCH 1/1] move exit from module_close to main |
Date: |
Thu, 18 Nov 2010 12:27:21 -0600 |
On Thu, Nov 18, 2010 at 01:27:43PM +0000, Christopher Brannon wrote:
> Andrei Kholodnyi <andrei.kholodnyi at gmail.com> writes:
>
> > + return(0);
>
> This is being very picky on my part. Most of the code uses the form
> return 0;,
> but you have consistently used return(0) throughout this patch. Should
> I change it? Does it really matter to anyone?
I would say that since return isn't a function call it shouldn't have
the parentheses. also, I would vote for changing it to make it
consistent with the rest of the code.
William
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
URL:
<http://lists.freebsoft.org/pipermail/speechd/attachments/20101118/8e3671a6/attachment.pgp>