Skip to content

Commit

Permalink
Dom: dead-loop req
Browse files Browse the repository at this point in the history
  • Loading branch information
fchn289 committed Mar 1, 2024
1 parent 3c7e39b commit 4865c5e
Showing 1 changed file with 9 additions and 7 deletions.
16 changes: 9 additions & 7 deletions src/domino/Domino.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -17,9 +17,6 @@
// - assmuption:
// . each event-hdlr is called only when event state F->T
// . repeated event/hdlr is complex, be careful(eg DominoTests.newTriggerViaChain)
// . dead-loop: complex so
// . not check in normal code
// . but via standalone func - deeperLinkThan()
//
// - core: states_
//
Expand All @@ -33,9 +30,15 @@
// . no use-up mem which is impossible in most cases
// . user shall not loop link ev - hard, expensive & unreasonable
// - MT safe: no
// - class safe: yes (all safe include mem-safe, no-ev-link-loop, no exception, etc, exclude MT safe)
// . ev-link-loop:
// . REQ:
// . no-loop to ensure safe
// . runtime forbid (rather than offline/afterward check which is not safe)
// . so fail setPrev() if loop
// . as simple as possible so little impact runtime (debug can be HID_CODE)
// ***********************************************************************************************
#ifndef DOMINO_HPP_
#define DOMINO_HPP_
#pragma once

#include <map>
#include <set>
Expand Down Expand Up @@ -109,7 +112,6 @@ class Domino : public UniLog
};

} // namespace
#endif // DOMINO_HPP_
// ***********************************************************************************************
// YYYY-MM-DD Who v)Modification Description
// .......... ......... .......................................................................
Expand Down Expand Up @@ -151,7 +153,7 @@ class Domino : public UniLog
// 2022-08-18 CSZ - replace CppLog by UniLog
// 2023-01-11 CSZ - search partial EvName
// 2023-11-14 CSZ 7)rm event
// 2024-02-29 CSZ - check dead loop
// 2024-02-29 CSZ 8)Dom is a all-safe class
// ***********************************************************************************************
// - where:
// . start using domino for time-cost events
Expand Down

0 comments on commit 4865c5e

Please sign in to comment.