|
From: | Paul Eggert |
Subject: | Re: [PATCH 08/34] maint: dfa: convert #if-MBS_SUPPORT (dfaoptimize, dfacomp) |
Date: | Thu, 15 Sep 2011 09:34:53 -0700 |
User-agent: | Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.2.21) Gecko/20110831 Thunderbird/3.1.13 |
After that patch, if I'm reading things correctly, dfaoptimize's body is of the form "if (MBS_SUPPORT) { ... }" and the only call to dfaoptimize is of the form "if (MBS_SUPPORT) dfaoptimize(d);". One of the "if" tests can be removed. I'd remove the one inside "dfaoptimize" as that results in less indenting.
[Prev in Thread] | Current Thread | [Next in Thread] |