EPICS Controls Argonne National Laboratory

Experimental Physics and
Industrial Control System

2002  2003  2004  2005  2006  2007  2008  2009  2010  2011  2012  2013  2014  2015  2016  2017  2018  2019  <20202021  2022  2023  2024  Index 2002  2003  2004  2005  2006  2007  2008  2009  2010  2011  2012  2013  2014  2015  2016  2017  2018  2019  <20202021  2022  2023  2024 
<== Date ==> <== Thread ==>

Subject: Re: [Merge] ~dirk.zimoch/epics-base:CleanupWhitespace into epics-base:7.0
From: Marty Kraimer via Core-talk <core-talk at aps.anl.gov>
To: core-talk at aps.anl.gov
Date: Thu, 16 Apr 2020 07:17:24 -0400
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  <20202021  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  <20202021  2022  2023  2024 
ANJ, 17 Apr 2020 Valid HTML 4.01! · Home · News · About · Base · Modules · Extensions · Distributions · Download ·
· Search · EPICS V4 · IRMIS · Talk · Bugs · Documents · Links · Licensing ·