In Dirk's message the following appears:
if (i<10)
doThis(i);
else
{
I have seen this cause problems when
doThis(i);
is changed from a single statement to multiple statements.
Better is
if (i<10) {
doThis(i);
} else {
...
}
Marty
On 4/16/20 3:42 AM, Dirk Zimoch via Core-talk wrote:
What is your favorite coding style, mine looks like this:
int function(int a, int b)
{
int i;
for (i=a; i<b; i++)
{
if (i<10)
doThis(i);
else
{
switch (i)
{
case 42:
doSomethingSpecial();
break;
default:
{
doTheNormalThing(i, a, b);
break;
}
}
}
}
return 0;
}
- Replies:
- AW: [Merge] ~dirk.zimoch/epics-base:CleanupWhitespace into epics-base:7.0 Zimoch Dirk (PSI) via Core-talk
- References:
- Re: [Merge] ~dirk.zimoch/epics-base:CleanupWhitespace into epics-base:7.0 Dirk Zimoch via Core-talk
- Navigate by Date:
- Prev:
Jenkins build is still unstable: epics-base-3.15-win32-test #290 APS Jenkins via Core-talk
- Next:
Build failed: epics-base base-fix-epicsFindSymbol-463 AppVeyor via Core-talk
- Index:
2002
2003
2004
2005
2006
2007
2008
2009
2010
2011
2012
2013
2014
2015
2016
2017
2018
2019
<2020>
2021
2022
2023
2024
- Navigate by Thread:
- Prev:
Re: [Merge] ~dirk.zimoch/epics-base:CleanupWhitespace into epics-base:7.0 Dirk Zimoch via Core-talk
- Next:
AW: [Merge] ~dirk.zimoch/epics-base:CleanupWhitespace into epics-base:7.0 Zimoch Dirk (PSI) via Core-talk
- Index:
2002
2003
2004
2005
2006
2007
2008
2009
2010
2011
2012
2013
2014
2015
2016
2017
2018
2019
<2020>
2021
2022
2023
2024
|