Skip to content

[Spreadsheet] Rename available columns after adjusting the number of columns

Ahmad Samir requested to merge (removed):l-spreadsheet into master

Move the loop that renames the currently available columns after the code that removes superfluous columns / adds new columns as needed. This fixes a crash when the available columns number is bigger than the size of colNameList (out-of-bounds index).

Also add a Q_ASSERT() to catch that issue in debug builds.

Incidentally this makes the asciifilter unit test pass again.

Merge request reports