unneeded k for loop

[gscteam]

Hotsport identified in perf top, It seems that the k-loop is not needed there.

[drrtuy]

Agree, this loop could be avoided. Could you post license-mantra?

[gscteam]

Submitted under the BSD license.

[gscteam]

There is one condition that might be missing, if columnsUpdated.size() is 0, then nothing get done. It is unclear for me when would that be the case, since the j loop for (unsigned int j = 0; j < columnsUpdated.size(); j++) is also based on that. So if that size is 0 it would not even reach that far.

[drrtuy]

Thanks for commenting this. We will look into this. чт, 9 мая 2019 г., 1:15 gscteam <notifications@github.com>:

…

There is one condition that might be missing, if columnsUpdated.size() is 0, then nothing get done. It is unclear for me when would that be the case, so I will let you add this condition if necessary. — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#746 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AC3JDLGZDPUK3Q4A7NAFJFLPUNGI5ANCNFSM4HJUIWCQ> .