search

coin-or / qpOASES

Open-source C++ implementation of the recently proposed online active set strategy
GNU Lesser General Public License v2.1
397 stars 132 forks source link

Is this ; intended? #135

Open ichumuh opened 2 years ago

ichumuh commented 2 years ago

Hi, I stumbled across this line: https://github.com/coin-or/qpOASES/blob/9bd3374a26d8e92e90eef1854228d73326dc90a2/src/Matrices.cpp#L853 Could it be that this ; at the end of the for loop was not intended?

HenryHwong commented 6 months ago

I found this problem, too. Further more, a lot of places in class SparseMatrix got a code style like for(...); statement; (15 counts) or for(...) {}; statement; (2 counts) I guess this may not be intended.