[checkmk-commits] [Checkmk/checkmk] f21925: Tiny simplifications.

Branch: refs/heads/master Home: https://github.com/Checkmk/checkmk Commit: f219255bceed00a1c3d8bbfce0c053fe8f5e4f99 https://github.com/Checkmk/checkmk/commit/f219255bceed00a1c3d8bbfce0c053fe8… Author: Sven Panne <sven.panne(a)checkmk.com> Date: 2024-04-30 (Tue, 30 Apr 2024) Changed paths: M cmk/gui/graphing/_utils.py M packages/cmk-werks/cmk/werks/parse.py Log Message: ----------- Tiny simplifications. Change-Id: I6b99d3440322013f5e749dd36a321a5175abf3da Commit: b3031e56a2830434dee0f2b0810c82e698e0378f https://github.com/Checkmk/checkmk/commit/b3031e56a2830434dee0f2b0810c82e69… Author: Sven Panne <sven.panne(a)checkmk.com> Date: 2024-04-30 (Tue, 30 Apr 2024) Changed paths: M agents/plugins/apache_status.py M agents/plugins/nginx_status.py M cmk/checkengine/parser/_agent.py M cmk/gui/graphing/_utils.py M cmk/gui/mkeventd/wato.py M packages/cmk-werks/cmk/werks/parse.py M tests/unit/cmk/base/plugins/agent_based/test_section_host_label_doc.py Log Message: ----------- Nano optimization for testing for a whitespace-only string. This avoids useless object creation. Normally not worth mentioning, but it is a) arguably even more readable and b) squeezes out a few percent in hot loops (e.g. section parsing) Change-Id: Ifb31dd582f4d39bfdc1f652006e99793b4094ab3 Commit: ba34466f8f89a12ff0d9cb73ae167453ddf8a5af https://github.com/Checkmk/checkmk/commit/ba34466f8f89a12ff0d9cb73ae167453d… Author: Sven Panne <sven.panne(a)checkmk.com> Date: 2024-04-30 (Tue, 30 Apr 2024) Changed paths: M cmk/checkengine/parser/_markers.py Log Message: ----------- Replaced ad hoc parsing with regex. The resulting code is shorter, a tiny bit faster and more consistent: With optimization switched on, asserts vanish. Change-Id: Ifd2290262cf18ee64f6cecaee53d5c70686da97e Commit: e18a4787ec0bb17a91544240f00edd89056033f6 https://github.com/Checkmk/checkmk/commit/e18a4787ec0bb17a91544240f00edd890… Author: Sven Panne <sven.panne(a)checkmk.com> Date: 2024-04-30 (Tue, 30 Apr 2024) Changed paths: M cmk/checkengine/parser/_agent.py Log Message: ----------- Nano optimization: Remove "hot" asserts. They are true by construction and cost about 7-8% of the whole section parsing time. Not very much, but one can't rely on the asserts, anyway: They might not be there when optimizations are enabled. Change-Id: I7866f379318226087bc562ec4ab7a1cd147ddb5f Compare: https://github.com/Checkmk/checkmk/compare/ecb67d3247ba...e18a4787ec0b To unsubscribe from these emails, change your notification settings at https://github.com/Checkmk/checkmk/settings/notifications