Refactor away C89 excess braces
-------------------------------

                 Key: LUCY-145
                 URL: https://issues.apache.org/jira/browse/LUCY-145
             Project: Lucy
          Issue Type: Sub-task
            Reporter: Marvin Humphrey
            Priority: Trivial


Because C89 does not allow mixed declarations and code, we have occasionally
used bare braces to create new scopes solely for the purpose of declaring a
variable or two.  In some cases, this was appropriate to limit the scope of a
short lived variable in the midst of a long function, but there were other
cases when we were just declaring a return value or such towards the end and
C89 forced the issue.

To find candidates for review, use the following command:

{noformat}
    grep -r "^ \+{ *$" lucy/ | grep -v -e "svn\|charmonizer\|modules\|\.pm"
{noformat}

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to