Comment out assignment in if statement.

As upstream is aware of the error, but does not want to correct it to
keep existing behaviour, we do the same. However, the assignment in the
if statement has no effect, since the variable is reassigned in the body
of the if statement. So we comment it out to prevent it from causing any
errors during compilation.

2 files changed, 32 insertions, 32 deletions

diff --git a/plugins/LinuxVST/src/Ditherbox/DitherboxProc.cpp b/plugins/LinuxVST/src/Ditherbox/DitherboxProc.cpp
index 4906413..1672f84 100755
--- a/plugins/LinuxVST/src/Ditherbox/DitherboxProc.cpp
+++ b/plugins/LinuxVST/src/Ditherbox/DitherboxProc.cpp
@@ -440,10 +440,10 @@ void Ditherbox::processReplacing(float **inputs, float **outputs, VstInt32 sampl
 				}
 				//assign the relevant one to the delay line
 				//and floor/ceil signal accordingly
-				
+
 				totalA = bynL[1] + bynL[2] + bynL[3] + bynL[4] + bynL[5] + bynL[6] + bynL[7] + bynL[8] + bynL[9];
 				totalA /= 1000;
-				if (totalA = 0) totalA = 1; // spotted by Laserbat: this 'scaling back' code doesn't. It always divides by the fallback of 1. Old NJAD doesn't scale back the things we're comparing against. Kept to retain known behavior, use the one in StudioTan and Monitoring for a tuned-as-intended NJAD.
+				/* if (totalA = 0) */ totalA = 1; // spotted by Laserbat: this 'scaling back' code doesn't. It always divides by the fallback of 1. Old NJAD doesn't scale back the things we're comparing against. Kept to retain known behavior, use the one in StudioTan and Monitoring for a tuned-as-intended NJAD.
 				bynL[1] /= totalA;
 				bynL[2] /= totalA;
 				bynL[3] /= totalA;
@@ -515,10 +515,10 @@ void Ditherbox::processReplacing(float **inputs, float **outputs, VstInt32 sampl
 				}
 				//assign the relevant one to the delay line
 				//and floor/ceil signal accordingly
-				
+
 				totalA = bynR[1] + bynR[2] + bynR[3] + bynR[4] + bynR[5] + bynR[6] + bynR[7] + bynR[8] + bynR[9];
 				totalA /= 1000;
-				if (totalA = 0) totalA = 1; // spotted by Laserbat: this 'scaling back' code doesn't. It always divides by the fallback of 1. Old NJAD doesn't scale back the things we're comparing against. Kept to retain known behavior, use the one in StudioTan and Monitoring for a tuned-as-intended NJAD.
+				/* if (totalA = 0) */ totalA = 1; // spotted by Laserbat: this 'scaling back' code doesn't. It always divides by the fallback of 1. Old NJAD doesn't scale back the things we're comparing against. Kept to retain known behavior, use the one in StudioTan and Monitoring for a tuned-as-intended NJAD.
 				bynR[1] /= totalA;
 				bynR[2] /= totalA;
 				bynR[3] /= totalA;
@@ -599,10 +599,10 @@ void Ditherbox::processReplacing(float **inputs, float **outputs, VstInt32 sampl
 				}
 				//assign the relevant one to the delay line
 				//and floor/ceil signal accordingly
-				
+
 				totalA = bynL[1] + bynL[2] + bynL[3] + bynL[4] + bynL[5] + bynL[6] + bynL[7] + bynL[8] + bynL[9];
 				totalA /= 1000;
-				if (totalA = 0) totalA = 1; // spotted by Laserbat: this 'scaling back' code doesn't. It always divides by the fallback of 1. Old NJAD doesn't scale back the things we're comparing against. Kept to retain known behavior, use the one in StudioTan and Monitoring for a tuned-as-intended NJAD.
+				/* if (totalA = 0) */ totalA = 1; // spotted by Laserbat: this 'scaling back' code doesn't. It always divides by the fallback of 1. Old NJAD doesn't scale back the things we're comparing against. Kept to retain known behavior, use the one in StudioTan and Monitoring for a tuned-as-intended NJAD.
 				bynL[1] /= totalA;
 				bynL[2] /= totalA;
 				bynL[3] /= totalA;
@@ -680,10 +680,10 @@ void Ditherbox::processReplacing(float **inputs, float **outputs, VstInt32 sampl
 				}
 				//assign the relevant one to the delay line
 				//and floor/ceil signal accordingly
-				
+
 				totalA = bynR[1] + bynR[2] + bynR[3] + bynR[4] + bynR[5] + bynR[6] + bynR[7] + bynR[8] + bynR[9];
 				totalA /= 1000;
-				if (totalA = 0) totalA = 1; // spotted by Laserbat: this 'scaling back' code doesn't. It always divides by the fallback of 1. Old NJAD doesn't scale back the things we're comparing against. Kept to retain known behavior, use the one in StudioTan and Monitoring for a tuned-as-intended NJAD.
+				/* if (totalA = 0) */ totalA = 1; // spotted by Laserbat: this 'scaling back' code doesn't. It always divides by the fallback of 1. Old NJAD doesn't scale back the things we're comparing against. Kept to retain known behavior, use the one in StudioTan and Monitoring for a tuned-as-intended NJAD.
 				bynR[1] /= totalA;
 				bynR[2] /= totalA;
 				bynR[3] /= totalA;
@@ -1444,10 +1444,10 @@ void Ditherbox::processDoubleReplacing(double **inputs, double **outputs, VstInt
 				}
 				//assign the relevant one to the delay line
 				//and floor/ceil signal accordingly
-				
+
 				totalA = bynL[1] + bynL[2] + bynL[3] + bynL[4] + bynL[5] + bynL[6] + bynL[7] + bynL[8] + bynL[9];
 				totalA /= 1000;
-				if (totalA = 0) totalA = 1; // spotted by Laserbat: this 'scaling back' code doesn't. It always divides by the fallback of 1. Old NJAD doesn't scale back the things we're comparing against. Kept to retain known behavior, use the one in StudioTan and Monitoring for a tuned-as-intended NJAD.
+				/* if (totalA = 0) */ totalA = 1; // spotted by Laserbat: this 'scaling back' code doesn't. It always divides by the fallback of 1. Old NJAD doesn't scale back the things we're comparing against. Kept to retain known behavior, use the one in StudioTan and Monitoring for a tuned-as-intended NJAD.
 				bynL[1] /= totalA;
 				bynL[2] /= totalA;
 				bynL[3] /= totalA;
@@ -1519,10 +1519,10 @@ void Ditherbox::processDoubleReplacing(double **inputs, double **outputs, VstInt
 				}
 				//assign the relevant one to the delay line
 				//and floor/ceil signal accordingly
-				
+
 				totalA = bynR[1] + bynR[2] + bynR[3] + bynR[4] + bynR[5] + bynR[6] + bynR[7] + bynR[8] + bynR[9];
 				totalA /= 1000;
-				if (totalA = 0) totalA = 1; // spotted by Laserbat: this 'scaling back' code doesn't. It always divides by the fallback of 1. Old NJAD doesn't scale back the things we're comparing against. Kept to retain known behavior, use the one in StudioTan and Monitoring for a tuned-as-intended NJAD.
+				/* if (totalA = 0) */ totalA = 1; // spotted by Laserbat: this 'scaling back' code doesn't. It always divides by the fallback of 1. Old NJAD doesn't scale back the things we're comparing against. Kept to retain known behavior, use the one in StudioTan and Monitoring for a tuned-as-intended NJAD.
 				bynR[1] /= totalA;
 				bynR[2] /= totalA;
 				bynR[3] /= totalA;
@@ -1603,10 +1603,10 @@ void Ditherbox::processDoubleReplacing(double **inputs, double **outputs, VstInt
 				}
 				//assign the relevant one to the delay line
 				//and floor/ceil signal accordingly
-				
+
 				totalA = bynL[1] + bynL[2] + bynL[3] + bynL[4] + bynL[5] + bynL[6] + bynL[7] + bynL[8] + bynL[9];
 				totalA /= 1000;
-				if (totalA = 0) totalA = 1; // spotted by Laserbat: this 'scaling back' code doesn't. It always divides by the fallback of 1. Old NJAD doesn't scale back the things we're comparing against. Kept to retain known behavior, use the one in StudioTan and Monitoring for a tuned-as-intended NJAD.
+				/* if (totalA = 0) */ totalA = 1; // spotted by Laserbat: this 'scaling back' code doesn't. It always divides by the fallback of 1. Old NJAD doesn't scale back the things we're comparing against. Kept to retain known behavior, use the one in StudioTan and Monitoring for a tuned-as-intended NJAD.
 				bynL[1] /= totalA;
 				bynL[2] /= totalA;
 				bynL[3] /= totalA;
@@ -1684,10 +1684,10 @@ void Ditherbox::processDoubleReplacing(double **inputs, double **outputs, VstInt
 				}
 				//assign the relevant one to the delay line
 				//and floor/ceil signal accordingly
-				
+
 				totalA = bynR[1] + bynR[2] + bynR[3] + bynR[4] + bynR[5] + bynR[6] + bynR[7] + bynR[8] + bynR[9];
 				totalA /= 1000;
-				if (totalA = 0) totalA = 1; // spotted by Laserbat: this 'scaling back' code doesn't. It always divides by the fallback of 1. Old NJAD doesn't scale back the things we're comparing against. Kept to retain known behavior, use the one in StudioTan and Monitoring for a tuned-as-intended NJAD.
+				/* if (totalA = 0) */ totalA = 1; // spotted by Laserbat: this 'scaling back' code doesn't. It always divides by the fallback of 1. Old NJAD doesn't scale back the things we're comparing against. Kept to retain known behavior, use the one in StudioTan and Monitoring for a tuned-as-intended NJAD.
 				bynR[1] /= totalA;
 				bynR[2] /= totalA;
 				bynR[3] /= totalA;
diff --git a/plugins/WinVST/Ditherbox/DitherboxProc.cpp b/plugins/WinVST/Ditherbox/DitherboxProc.cpp
index 4906413..262b499 100755
--- a/plugins/WinVST/Ditherbox/DitherboxProc.cpp
+++ b/plugins/WinVST/Ditherbox/DitherboxProc.cpp
@@ -440,10 +440,10 @@ void Ditherbox::processReplacing(float **inputs, float **outputs, VstInt32 sampl
 				}
 				//assign the relevant one to the delay line
 				//and floor/ceil signal accordingly
-				
+
 				totalA = bynL[1] + bynL[2] + bynL[3] + bynL[4] + bynL[5] + bynL[6] + bynL[7] + bynL[8] + bynL[9];
 				totalA /= 1000;
-				if (totalA = 0) totalA = 1; // spotted by Laserbat: this 'scaling back' code doesn't. It always divides by the fallback of 1. Old NJAD doesn't scale back the things we're comparing against. Kept to retain known behavior, use the one in StudioTan and Monitoring for a tuned-as-intended NJAD.
+				/* /* if (totalA = 0) */ */ totalA = 1; // spotted by Laserbat: this 'scaling back' code doesn't. It always divides by the fallback of 1. Old NJAD doesn't scale back the things we're comparing against. Kept to retain known behavior, use the one in StudioTan and Monitoring for a tuned-as-intended NJAD.
 				bynL[1] /= totalA;
 				bynL[2] /= totalA;
 				bynL[3] /= totalA;
@@ -515,10 +515,10 @@ void Ditherbox::processReplacing(float **inputs, float **outputs, VstInt32 sampl
 				}
 				//assign the relevant one to the delay line
 				//and floor/ceil signal accordingly
-				
+
 				totalA = bynR[1] + bynR[2] + bynR[3] + bynR[4] + bynR[5] + bynR[6] + bynR[7] + bynR[8] + bynR[9];
 				totalA /= 1000;
-				if (totalA = 0) totalA = 1; // spotted by Laserbat: this 'scaling back' code doesn't. It always divides by the fallback of 1. Old NJAD doesn't scale back the things we're comparing against. Kept to retain known behavior, use the one in StudioTan and Monitoring for a tuned-as-intended NJAD.
+				/* if (totalA = 0) */ totalA = 1; // spotted by Laserbat: this 'scaling back' code doesn't. It always divides by the fallback of 1. Old NJAD doesn't scale back the things we're comparing against. Kept to retain known behavior, use the one in StudioTan and Monitoring for a tuned-as-intended NJAD.
 				bynR[1] /= totalA;
 				bynR[2] /= totalA;
 				bynR[3] /= totalA;
@@ -599,10 +599,10 @@ void Ditherbox::processReplacing(float **inputs, float **outputs, VstInt32 sampl
 				}
 				//assign the relevant one to the delay line
 				//and floor/ceil signal accordingly
-				
+
 				totalA = bynL[1] + bynL[2] + bynL[3] + bynL[4] + bynL[5] + bynL[6] + bynL[7] + bynL[8] + bynL[9];
 				totalA /= 1000;
-				if (totalA = 0) totalA = 1; // spotted by Laserbat: this 'scaling back' code doesn't. It always divides by the fallback of 1. Old NJAD doesn't scale back the things we're comparing against. Kept to retain known behavior, use the one in StudioTan and Monitoring for a tuned-as-intended NJAD.
+				/* if (totalA = 0) */ totalA = 1; // spotted by Laserbat: this 'scaling back' code doesn't. It always divides by the fallback of 1. Old NJAD doesn't scale back the things we're comparing against. Kept to retain known behavior, use the one in StudioTan and Monitoring for a tuned-as-intended NJAD.
 				bynL[1] /= totalA;
 				bynL[2] /= totalA;
 				bynL[3] /= totalA;
@@ -680,10 +680,10 @@ void Ditherbox::processReplacing(float **inputs, float **outputs, VstInt32 sampl
 				}
 				//assign the relevant one to the delay line
 				//and floor/ceil signal accordingly
-				
+
 				totalA = bynR[1] + bynR[2] + bynR[3] + bynR[4] + bynR[5] + bynR[6] + bynR[7] + bynR[8] + bynR[9];
 				totalA /= 1000;
-				if (totalA = 0) totalA = 1; // spotted by Laserbat: this 'scaling back' code doesn't. It always divides by the fallback of 1. Old NJAD doesn't scale back the things we're comparing against. Kept to retain known behavior, use the one in StudioTan and Monitoring for a tuned-as-intended NJAD.
+				/* if (totalA = 0) */ totalA = 1; // spotted by Laserbat: this 'scaling back' code doesn't. It always divides by the fallback of 1. Old NJAD doesn't scale back the things we're comparing against. Kept to retain known behavior, use the one in StudioTan and Monitoring for a tuned-as-intended NJAD.
 				bynR[1] /= totalA;
 				bynR[2] /= totalA;
 				bynR[3] /= totalA;
@@ -1444,10 +1444,10 @@ void Ditherbox::processDoubleReplacing(double **inputs, double **outputs, VstInt
 				}
 				//assign the relevant one to the delay line
 				//and floor/ceil signal accordingly
-				
+
 				totalA = bynL[1] + bynL[2] + bynL[3] + bynL[4] + bynL[5] + bynL[6] + bynL[7] + bynL[8] + bynL[9];
 				totalA /= 1000;
-				if (totalA = 0) totalA = 1; // spotted by Laserbat: this 'scaling back' code doesn't. It always divides by the fallback of 1. Old NJAD doesn't scale back the things we're comparing against. Kept to retain known behavior, use the one in StudioTan and Monitoring for a tuned-as-intended NJAD.
+				/* if (totalA = 0) */ totalA = 1; // spotted by Laserbat: this 'scaling back' code doesn't. It always divides by the fallback of 1. Old NJAD doesn't scale back the things we're comparing against. Kept to retain known behavior, use the one in StudioTan and Monitoring for a tuned-as-intended NJAD.
 				bynL[1] /= totalA;
 				bynL[2] /= totalA;
 				bynL[3] /= totalA;
@@ -1519,10 +1519,10 @@ void Ditherbox::processDoubleReplacing(double **inputs, double **outputs, VstInt
 				}
 				//assign the relevant one to the delay line
 				//and floor/ceil signal accordingly
-				
+
 				totalA = bynR[1] + bynR[2] + bynR[3] + bynR[4] + bynR[5] + bynR[6] + bynR[7] + bynR[8] + bynR[9];
 				totalA /= 1000;
-				if (totalA = 0) totalA = 1; // spotted by Laserbat: this 'scaling back' code doesn't. It always divides by the fallback of 1. Old NJAD doesn't scale back the things we're comparing against. Kept to retain known behavior, use the one in StudioTan and Monitoring for a tuned-as-intended NJAD.
+				/* if (totalA = 0) */ totalA = 1; // spotted by Laserbat: this 'scaling back' code doesn't. It always divides by the fallback of 1. Old NJAD doesn't scale back the things we're comparing against. Kept to retain known behavior, use the one in StudioTan and Monitoring for a tuned-as-intended NJAD.
 				bynR[1] /= totalA;
 				bynR[2] /= totalA;
 				bynR[3] /= totalA;
@@ -1603,10 +1603,10 @@ void Ditherbox::processDoubleReplacing(double **inputs, double **outputs, VstInt
 				}
 				//assign the relevant one to the delay line
 				//and floor/ceil signal accordingly
-				
+
 				totalA = bynL[1] + bynL[2] + bynL[3] + bynL[4] + bynL[5] + bynL[6] + bynL[7] + bynL[8] + bynL[9];
 				totalA /= 1000;
-				if (totalA = 0) totalA = 1; // spotted by Laserbat: this 'scaling back' code doesn't. It always divides by the fallback of 1. Old NJAD doesn't scale back the things we're comparing against. Kept to retain known behavior, use the one in StudioTan and Monitoring for a tuned-as-intended NJAD.
+				/* if (totalA = 0) */ totalA = 1; // spotted by Laserbat: this 'scaling back' code doesn't. It always divides by the fallback of 1. Old NJAD doesn't scale back the things we're comparing against. Kept to retain known behavior, use the one in StudioTan and Monitoring for a tuned-as-intended NJAD.
 				bynL[1] /= totalA;
 				bynL[2] /= totalA;
 				bynL[3] /= totalA;
@@ -1684,10 +1684,10 @@ void Ditherbox::processDoubleReplacing(double **inputs, double **outputs, VstInt
 				}
 				//assign the relevant one to the delay line
 				//and floor/ceil signal accordingly
-				
+
 				totalA = bynR[1] + bynR[2] + bynR[3] + bynR[4] + bynR[5] + bynR[6] + bynR[7] + bynR[8] + bynR[9];
 				totalA /= 1000;
-				if (totalA = 0) totalA = 1; // spotted by Laserbat: this 'scaling back' code doesn't. It always divides by the fallback of 1. Old NJAD doesn't scale back the things we're comparing against. Kept to retain known behavior, use the one in StudioTan and Monitoring for a tuned-as-intended NJAD.
+				/* if (totalA = 0) */ totalA = 1; // spotted by Laserbat: this 'scaling back' code doesn't. It always divides by the fallback of 1. Old NJAD doesn't scale back the things we're comparing against. Kept to retain known behavior, use the one in StudioTan and Monitoring for a tuned-as-intended NJAD.
 				bynR[1] /= totalA;
 				bynR[2] /= totalA;
 				bynR[3] /= totalA;