[TC 11.00b1 32-bit] Custom column header 32 fails if loaded at startup

[white]

* Add this custom columns view "test" (32 empty columns, last 2 columns have header "31" and "32", horizontal scrollbar enabled)

Code: Select all

[CustomFields]
Widths1=80,30,15,15,15,15,15,15,15,15,15,15,15,15,15,15,15,15,15,15,15,15,15,15,15,15,15,15,15,15,15,15,15,15
Headers1=\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n31\n32
Contents1=\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n
Options1=-1|1|96
AutoLoad=0
Titles=test

* Run TC 32-bit
* Press Shift+F1 and select the custom column "test"
* Scroll to the right so the headers of custom column 31 and 32 come into view
* Execute: cm_exit 1 (or close en re-open TC)
* Scroll to the right so the headers of custom column 31 and 32 come into view (custom column headers 31 and 32 are wrong)
* Press Shift+F1 and select the custom column "test" (custom column headers 31 and 32 are good)

To see the difference between 32-bit and 64-bit:
* Execute: cm_exit 2 (custom column headers 31 and 32 are good in TC 64-bit)
* Execute: cm_exit 2 (custom column headers 31 and 32 are wrong in TC 32-bit)

[ghisler(Author)]

Isn't the 64-bit version wrong?
Here is the fix:
25.01.23 Fixed: Custom columns view: Enforce maximum number of 32 columns (in addition to name and extension) when loading and configuring custom columns (32/64)
With name and ext, there are 32 valid columns.

[white]

Isn't the 64-bit version wrong?

No.

25.01.23 Fixed: Custom columns view: Enforce maximum number of 32 columns (in addition to name and extension) when loading and configuring custom columns (32/64)
With name and ext, there are 32 valid columns.

No? You just quoted from HISTORY.TXT that there are 32 valid custom columns in addition to name and extension, so 34 in total. Did you mean to limit the total number of columns to 32? In that case TC 32-bit and TC 64-bit are both wrong.

[ghisler(Author)]

Odd, I had in mind that there were 32 columns in total. I will check it.

[white]

Since you mean 32 in total, you may also want to replace the text "in addition to" with "including".

[ghisler(Author)]

There are indeed 32 additional columns, there was just an error in the code...

[white]

Tested OK using TC 11.00 b2 32-bit.

Moderator message from: white » 2023-05-10, 17:00 UTC

Moved to Fixed bugs.