Community
Participate
Working Groups
Created attachment 186357 [details] suggested fix similar to other bugs related to bug 333678 ... This case is a little complicated, since the only one detected as unused by compiler is 'width' but if assuming none of the "table.getWidth()" type of method have side effects, there's a whole chunk of code that's not needed. This might be a case where the "unused variable" is really detecting a problem in the code logic? Hard for me to tell. But in my suggested fix, I've just removed all the seemingly unused code.
David, this was actually an unnecessary code. I removed it and committed the change into HEAD. Let me know if it is also needed for 3.2.3.
I would prefer it in 3.2.3. We might not change the compiler in the 3.2.3 stream (normally we would not, and, surely will not right now) ... but, if 3.2.x turns out to be a long term maintenance stream then we might eventually also want to change the compiler, so then we would go through all this over again ... or, you can wait until it actually shows up? Up to you ... depending on ease/safety, etc. Thanks for handling.
Thanks for the clarification, David. I committed the change also in the 3.2.3 branch.
