• last updated 14 hours ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Prefer unset to array unset when the whole array should be deleted

See https://wiki.tcl-lang.org/page/Dict+VS+Array+Speed

    • -2
    • +2
    /openacs-4/packages/search/tcl/search-procs.tcl
    • -2
    • +2
    /openacs-4/packages/xowf/tcl/xowf-procs.tcl
Prefer unset to array unset when the whole array should be deleted, rework array idioms into tcl idioms when an array is not needed

See https://wiki.tcl-lang.org/page/Dict+VS+Array+Speed

Prefer unset to array unset when we do not need a pattern match

See https://wiki.tcl-lang.org/page/Dict+VS+Array+Speed

Rework so that an array is not needed

Prefer unset to array unset when the whole array should be deleted

See https://wiki.tcl-lang.org/page/Dict+VS+Array+Speed

Rework array idioms

Modernize tcl idioms

Prefer unset to array unset when the whole array should be deleted

See https://wiki.tcl-lang.org/page/Dict+VS+Array+Speed

    • -1
    • +1
    /openacs-4/packages/caldav/tcl/caldav-procs.tcl
Prefer unset to array unset when the whole array should be deleted, rework array idioms into dict idioms to avoid unsetting the array, where appropriate

See https://wiki.tcl-lang.org/page/Dict+VS+Array+Speed

Prefer unset to array unset when the whole array should be deleted

See https://wiki.tcl-lang.org/page/Dict+VS+Array+Speed

Rework array idioms in dict idioms, closer to the intended logics

Cleanup include found to not be in use in upstream codebase already in 2019

Rework array idioms into dict idioms to avoid the need to unset

Prefer unset to array unset when the whole array should be deleted

See https://wiki.tcl-lang.org/page/Dict+VS+Array+Speed

Streamline idiom

Prefer unset to array unset when the purpose is to delete the whole array

See https://wiki.tcl-lang.org/page/Dict+VS+Array+Speed

Prefer unset to array unset when we want to delete the entire array

See https://wiki.tcl-lang.org/page/Dict+VS+Array+Speed

Prefer unset to array unset when the semantic is to delete the whole array

See https://wiki.tcl-lang.org/page/Dict+VS+Array+Speed

State Highcharts package dependency in xotcl-request-monitor

Other leftovers

    • -30
    • +0
    /openacs-4/packages/chat/www/transcript-edit-2.tcl
Other files deleted

Delete more zombie files

    • -24
    • +0
    /openacs-4/packages/chat/www/moderator-grant-2.tcl
Delete zombie file from merge with MAIN

Trap illegal encoding exception thrown by tcl9 when a file contains invalid UTF-8 and skip the package with a warning

Unmaintained packages may contain illegal characters and we do not want the UI to crash in this situation

Limit files__tcl_file_common_errors test to installed packages

Unmaintained packages may ship files with invalid encodings, which tcl9 won't tolerate anymore.

Improve test

- skip packages that are not installed before we check the info file

- ensure all installed packages have an info file

This fixes the test for tcl9, where some unmaintained packages may fail the file parsing because of invalid encoding.

Tcl9 regression test: deactivated case broken in tcl9 with unclear semantics

whitespace change

tcl9 changes in regression test: "2000-00-00 00:00:00" is not a valid date anymore

tcl9 changes: "string bytelength" does not exist in tcl9