Branch: refs/heads/master Home: https://github.com/Checkmk/checkmk Commit: 2e126f64e894fb173b22af7f98ecc3432664ee46 https://github.com/Checkmk/checkmk/commit/2e126f64e894fb173b22af7f98ecc3432… Author: Sven Panne <sven.panne(a)checkmk.com> Date: 2024-10-18 (Fri, 18 Oct 2024) Changed paths: M packages/livestatus/include/livestatus/TableStateHistory.h M packages/livestatus/src/TableStateHistory.cc Log Message: ----------- Const-correctness Change-Id: I4e7a18270b3acd7fab095b339acc441e247ae4c8 Commit: 694cb99ace8baa47878f9a4bed7ae3f2424f2a1d https://github.com/Checkmk/checkmk/commit/694cb99ace8baa47878f9a4bed7ae3f24… Author: Sven Panne <sven.panne(a)checkmk.com> Date: 2024-10-18 (Fri, 18 Oct 2024) Changed paths: M packages/livestatus/include/livestatus/TableStateHistory.h M packages/livestatus/src/TableStateHistory.cc Log Message: ----------- Boil down TableStateHistory::handle_state_change()'s argument list a bit Using a boolean argument is an anti-pattern most of the time, but using a single boolean instead of 3 (derived) arguments seems like a good tradeoff. In any case, we can clean this up later if wanted/needed. Change-Id: I520478bfd7537436dd88d2ee90e85ef9f1c1a5e3 Compare: https://github.com/Checkmk/checkmk/compare/bb9ebfd2382e...694cb99ace8b To unsubscribe from these emails, change your notification settings at https://github.com/Checkmk/checkmk/settings/notifications