[us-commits] [ehb54/ultrascan3] c7ebdd: Fix "No such signal QHeaderView::released(int)" wa...

[emre brookes]

  Branch: refs/heads/somo-dev-claude
  Home:   https://github.com/ehb54/ultrascan3
  Commit: c7ebdda2f2175331c0231fbc2fe4e7cc9eebbeda
      https://github.com/ehb54/ultrascan3/commit/c7ebdda2f2175331c0231fbc2fe4e7cc9eebbeda
  Author: ehb54 <brookes at uthscsa.edu>
  Date:   2026-06-21 (Sun, 21 Jun 2026)

  Changed paths:
    M us_somo/develop/src/us_hydrodyn_dad_conc.cpp
    M us_somo/develop/src/us_hydrodyn_dad_conc_load.cpp
    M us_somo/develop/src/us_hydrodyn_mals_conc.cpp
    M us_somo/develop/src/us_hydrodyn_mals_conc_load.cpp
    M us_somo/develop/src/us_hydrodyn_mals_saxs_conc.cpp
    M us_somo/develop/src/us_hydrodyn_mals_saxs_conc_load.cpp
    M us_somo/develop/src/us_hydrodyn_saxs_buffer_conc.cpp
    M us_somo/develop/src/us_hydrodyn_saxs_buffer_conc_load.cpp
    M us_somo/develop/src/us_hydrodyn_saxs_hplc_conc.cpp
    M us_somo/develop/src/us_hydrodyn_saxs_hplc_conc_load.cpp

  Log Message:
  -----------
  Fix "No such signal QHeaderView::released(int)" warning in conc dialogs

QHeaderView::released(int) is a Qt3/Qt4 signal name that doesn't exist
in Qt5 (it's the column/row header click handler in the various
"Concentrations" table dialogs across DAD, MALS, MALS+SAXS, SAXS,
SAXS buffer, and SAXS Hplc/Kin). The connect() silently failed at
runtime with a console warning, and the header-click slot (col/row
_header_released) never fired.

us_hydrodyn_saxs_conc.cpp already had the correct fix, properly guarded
behind #if QT_VERSION < 0x040000 / #else sectionClicked(int) / #endif --
the other 10 copy-pasted variants of this dialog never got that fix and
had the broken signal name unconditionally. Replaced with
sectionClicked(int) (the Qt5 equivalent) in all of them, matching the
already-proven-working pattern.

Co-Authored-By: Claude Sonnet 4.6 <noreply at anthropic.com>



To unsubscribe from these emails, change your notification settings at https://github.com/ehb54/ultrascan3/settings/notifications