[yoshimi] Re: More firefighting :(

  • From: Kristian Amlie <kristian@xxxxxxxxxx>
  • To: yoshimi@xxxxxxxxxxxxx
  • Date: Thu, 11 Feb 2016 15:57:48 +0100

On 09. feb. 2016 00:03, Will Godfrey wrote:

I appears that yoshimi will fail on the upcoming gcc 6 compiler.

...

Is anyone experimenting with this yet? If so can give it a run through please 
to
see if I've got it right?

--- a/src/Effects/Alienwah.cpp
+++ b/src/Effects/Alienwah.cpp
@@ -70,7 +70,7 @@ void Alienwah::out(float *smpsl, float *smpsr)
         tmp = clfol * x + oldclfol * x1;
 
         out = tmp * oldl[oldk];
-        out.real() += (1 - abs(fb)) * smpsl[i] * pangainL;
+        out += (1 - abs(fb)) * smpsl[i] * pangainL;
 
         oldl[oldk] = out;
         float l = out.real() * 10.0f * (fb + 0.1f);
@@ -79,7 +79,7 @@ void Alienwah::out(float *smpsl, float *smpsr)
         tmp = clfor * x + oldclfor * x1;
 
         out = tmp * oldr[oldk];
-        out.real() += (1 - abs(fb)) * smpsr[i] * pangainR;
+        out += (1 - abs(fb)) * smpsr[i] * pangainR;
 
         oldr[oldk] = out;
         float r = out.real() * 10.0f * (fb + 0.1f);

AFAICS, all values you add are real values, so this should be both
programmatically and mathematically correct!

--- a/src/Misc/CmdInterface.cpp
+++ b/src/Misc/CmdInterface.cpp
@@ -228,7 +228,7 @@ void CmdInterface::helpLoop(list<string>& msg, string 
*commands, int indent)
     while (commands[word] != "end")
     {
         left = commands[word];
-        msg.push_back(dent.assign<int>(indent, ' ') + left + 
blanks.assign<int>(spaces - left.length(), ' ') + "- " + commands[word + 1]);
+        msg.push_back(dent.assign(indent, ' ') + left + blanks.assign(spaces 
- left.length(), ' ') + "- " + commands[word + 1]);
         word += 2;
     }
 }

To tell you the truth I don't understand how the old version ever worked
(probably a side effect of how the STL library is implemented in gcc),
so the new one is definitely more correct!

-- 
Kristian
Yoshimi source code is available from either: 
http://sourceforge.net/projects/yoshimi
Or: https://github.com/Yoshimi/yoshimi
Our list archive is at: https://www.freelists.org/archive/yoshimi
To post, email to yoshimi@xxxxxxxxxxxxx

Other related posts: