lf-lang / reactor-uc

A lightweight reactor runtime targeted at resource-constrained embedded systems
BSD 2-Clause "Simplified" License
3 stars 1 forks source link

Add DECLARE_CTOR macros #113

Closed erlingrj closed 2 weeks ago

erlingrj commented 3 weeks ago

We need to declare the ctors in header files and define them in source files

github-actions[bot] commented 3 weeks ago

Coverage after merging more-macro into main will be

72.17%

Coverage Report
FileStmtsBranchesFuncsLinesUncovered Lines
src
   action.c92.21%81.25%100%94.74%23, 33, 39–42
   builtin_triggers.c90.24%70%100%96.43%14, 18, 37, 40
   connection.c80.69%53.85%100%89.69%10, 101, 107, 11, 120–121, 133–134, 14, 14, 140, 142, 17–18, 18, 18–19, 21, 23–24, 29, 44, 47, 52, 57–59, 94
   environment.c88.46%83.33%83.33%90.74%28, 35–36, 76–78, 9
   event.c94.44%90%100%95.65%10, 4
   federated.c0%0%0%0%100–102, 102, 102–103, 103, 103–105, 107, 11, 110–111, 113–117, 119, 12, 120–124, 126, 126, 126–129, 131, 131, 131–133, 133, 133–134, 138–139, 139, 139, 14, 142–143, 147–149, 15, 151, 151, 151, 153–157, 16, 160, 160, 160–163, 166–167, 167, 167–168, 17, 170–171, 174–175, 180–181, 181, 181–182, 184, 186, 186, 186–189, 189, 189, 189, 189, 19, 19, 19, 190–199, 20, 20, 20, 200–201, 205, 208, 208, 208–210, 214, 217–218, 218, 218, 218–219, 22, 22, 22, 220–226, 228, 23, 230, 234–239, 24, 24, 24, 240–241, 245–246, 248–249, 25, 251–254, 256, 256, 256–258, 26, 260, 30–31, 35–42, 42, 42–43, 43, 43–44, 44, 44, 47–48, 50–53, 55, 55, 55–58, 60, 62, 64, 64, 64–65, 67–68, 68, 68–69, 71–72, 74, 78–79, 81–83, 86, 88–93, 95–97
   logging.c73.21%60%100%75%24, 24–27, 37–39, 46, 46–49, 59–60
   port.c90.91%58.33%100%100%10, 15, 19, 24–25
   queues.c90.74%80.77%100%94.90%102, 107, 113, 21–23, 46–47, 59–60, 83–87
   reaction.c90.41%75%100%97.78%18, 20, 24, 43–44, 54, 56
   reactor.c67.14%40%100%76.09%16, 19–20, 20, 20–21, 21, 21–22, 24, 38–39, 42–43, 43, 43–44, 44, 44–45, 47, 60–61
   scheduler.c81.15%64.77%94.12%87.02%102, 104, 104, 111, 125, 17, 174, 177, 177, 177–180, 182–183, 183, 183–184, 204, 224–225, 231–233, 27, 277–278, 282, 286–287, 305, 31, 36, 52–54, 54, 54–56, 56, 56, 58, 58, 58–59, 61–62, 62, 62–63, 69–72, 80–81, 85
   serialization.c58.33%75%50%56.25%16–17, 33–35, 38–40
   tag.c40.19%31.48%60%47.92%14, 14–15, 17, 17–18, 23–24, 24, 24, 24, 24–25, 27, 27, 27, 27, 27–28, 30, 30, 30–31, 33–34, 34, 34–35, 37, 37, 37, 37, 37–38, 40, 40, 40, 40, 40–41, 43, 53–54, 63, 63–64, 83–85, 85, 85, 85, 85, 85, 85, 85, 85, 85, 85–87, 89
   timer.c94.59%66.67%100%100%14, 25
   trigger.c100%100%100%100%
src/platform/posix
   posix.c79.41%55.56%85.71%84.29%100, 16, 18, 20–21, 34–36, 38–40, 48–49, 62, 67, 79, 82, 88, 94
   tcp_ip_channel.c62.39%50.93%85.71%66.38%103, 105, 105, 105–107, 109–110, 110, 110–111, 111, 111–112, 114–115, 117, 117, 117, 119–120, 122–123, 127, 129–130, 130, 130–131, 133, 133, 133–134, 136, 144, 151, 156–157, 162–165, 170–171, 171, 171–174, 174, 174–177, 179–182, 184, 184, 184–186, 188–191, 194, 222–224, 231, 236–238, 246, 246–248, 25–26, 26, 26–27, 274–275, 275, 275–276, 28, 299–300, 304–305, 316, 319, 32, 324, 33–34, 343–344, 347–348, 354–355, 368, 37, 370, 374–375, 38, 389, 39, 390, 394–395, 399–400, 60–61, 66–67, 71–72, 92, 92, 92, 96–97
github-actions[bot] commented 3 weeks ago

Memory usage after merging this PR will be:

Memory Report ## action_microstep_test_c | | from | to | increase (%) | | -- | ---- | ------ | ------------ | | text | 54417 | 54417 | 0.00 | | data | 752 | 752 | 0.00 | | bss | 480 | 480 | 0.00 | | total | 55649 | 55649 | 0.00 | ## action_test_c | | from | to | increase (%) | | -- | ---- | ------ | ------------ | | text | 54188 | 54188 | 0.00 | | data | 752 | 752 | 0.00 | | bss | 480 | 480 | 0.00 | | total | 55420 | 55420 | 0.00 | ## delayed_conn_test_c | | from | to | increase (%) | | -- | ---- | ------ | ------------ | | text | 54776 | 54776 | 0.00 | | data | 744 | 744 | 0.00 | | bss | 480 | 480 | 0.00 | | total | 56000 | 56000 | 0.00 | ## event_payload_pool_test_c | | from | to | increase (%) | | -- | ---- | ------ | ------------ | | text | 18297 | 18297 | 0.00 | | data | 624 | 624 | 0.00 | | bss | 320 | 320 | 0.00 | | total | 19241 | 19241 | 0.00 | ## event_queue_test_c | | from | to | increase (%) | | -- | ---- | ------ | ------------ | | text | 27239 | 27239 | 0.00 | | data | 728 | 728 | 0.00 | | bss | 480 | 480 | 0.00 | | total | 28447 | 28447 | 0.00 | ## nanopb_test_c | | from | to | increase (%) | | -- | ---- | ------ | ------------ | | text | 42884 | 42884 | 0.00 | | data | 904 | 904 | 0.00 | | bss | 320 | 320 | 0.00 | | total | 44108 | 44108 | 0.00 | ## physical_action_test_c | | from | to | increase (%) | | -- | ---- | ------ | ------------ | | text | 55381 | 55381 | 0.00 | | data | 769 | 769 | 0.00 | | bss | 10240 | 10240 | 0.00 | | total | 66390 | 66390 | 0.00 | ## port_test_c | | from | to | increase (%) | | -- | ---- | ------ | ------------ | | text | 54651 | 54651 | 0.00 | | data | 744 | 744 | 0.00 | | bss | 480 | 480 | 0.00 | | total | 55875 | 55875 | 0.00 | ## reaction_queue_test_c | | from | to | increase (%) | | -- | ---- | ------ | ------------ | | text | 26951 | 26951 | 0.00 | | data | 728 | 728 | 0.00 | | bss | 480 | 480 | 0.00 | | total | 28159 | 28159 | 0.00 | ## request_shutdown_test_c | | from | to | increase (%) | | -- | ---- | ------ | ------------ | | text | 54979 | 54979 | 0.00 | | data | 744 | 744 | 0.00 | | bss | 480 | 480 | 0.00 | | total | 56203 | 56203 | 0.00 | ## shutdown_test_c | | from | to | increase (%) | | -- | ---- | ------ | ------------ | | text | 51926 | 51926 | 0.00 | | data | 752 | 752 | 0.00 | | bss | 10912 | 10912 | 0.00 | | total | 63590 | 63590 | 0.00 | ## startup_test_c | | from | to | increase (%) | | -- | ---- | ------ | ------------ | | text | 51257 | 51257 | 0.00 | | data | 752 | 752 | 0.00 | | bss | 10688 | 10688 | 0.00 | | total | 62697 | 62697 | 0.00 | ## tcp_channel_test_c | | from | to | increase (%) | | -- | ---- | ------ | ------------ | | text | 58041 | 58041 | 0.00 | | data | 1176 | 1176 | 0.00 | | bss | 11072 | 11072 | 0.00 | | total | 70289 | 70289 | 0.00 | ## timer_test_c | | from | to | increase (%) | | -- | ---- | ------ | ------------ | | text | 51158 | 51158 | 0.00 | | data | 744 | 744 | 0.00 | | bss | 10720 | 10720 | 0.00 | | total | 62622 | 62622 | 0.00 |
erlingrj commented 2 weeks ago

This is actually not necessary