Skip to content

Commit fcd74c6

Browse files
committed
Remove ControlSlotIndicator and ControlSlotCompositeIndicator
We previously introduced these classes to avoid additional bounds checking done when indexing into a DynamicArray in shared memory. Since we now are using the local views into shared memory, we no longer need this optimisation and can revert to simply accessing slots using their indices.
1 parent bccd1b3 commit fcd74c6

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

42 files changed

+479
-1363
lines changed

score/mw/com/impl/bindings/lola/BUILD

Lines changed: 0 additions & 47 deletions
Original file line numberDiff line numberDiff line change
@@ -543,26 +543,6 @@ cc_library(
543543
],
544544
)
545545

546-
cc_library(
547-
name = "control_slot_indicator",
548-
srcs = [
549-
"control_slot_composite_indicator.cpp",
550-
"control_slot_indicator.cpp",
551-
],
552-
hdrs = [
553-
"control_slot_composite_indicator.h",
554-
"control_slot_indicator.h",
555-
],
556-
features = COMPILER_WARNING_FEATURES,
557-
tags = ["FFI"],
558-
visibility = ["//score/mw/com/impl/bindings/lola:__subpackages__"],
559-
deps = [
560-
":control_slot_types",
561-
":event_slot_status",
562-
"@score_baselibs//score/language/futurecpp",
563-
],
564-
)
565-
566546
cc_library(
567547
name = "event_data_control",
568548
srcs = ["event_data_control.cpp"],
@@ -571,7 +551,6 @@ cc_library(
571551
tags = ["FFI"],
572552
visibility = ["//score/mw/com/impl/bindings/lola:__subpackages__"],
573553
deps = [
574-
":control_slot_indicator",
575554
":control_slot_types",
576555
":event_slot_status",
577556
":transaction_log",
@@ -592,7 +571,6 @@ cc_library(
592571
visibility = ["//score/mw/com/impl/bindings/lola:__subpackages__"],
593572
deps = [
594573
":event_data_control",
595-
":control_slot_indicator",
596574
":control_slot_types",
597575
":event_slot_status",
598576
":transaction_log_set",
@@ -609,7 +587,6 @@ cc_library(
609587
visibility = ["//score/mw/com/impl/bindings/lola:__subpackages__"],
610588
deps = [
611589
":event_data_control",
612-
":control_slot_indicator",
613590
":control_slot_types",
614591
":event_slot_status",
615592
":transaction_log_set",
@@ -1218,28 +1195,6 @@ cc_gtest_unit_test(
12181195
],
12191196
)
12201197

1221-
cc_gtest_unit_test(
1222-
name = "control_slot_composite_indicator_test",
1223-
srcs = [
1224-
"control_slot_composite_indicator_test.cpp",
1225-
],
1226-
features = COMPILER_WARNING_FEATURES,
1227-
deps = [
1228-
":control_slot_indicator",
1229-
],
1230-
)
1231-
1232-
cc_gtest_unit_test(
1233-
name = "control_slot_indicator_test",
1234-
srcs = [
1235-
"control_slot_indicator_test.cpp",
1236-
],
1237-
features = COMPILER_WARNING_FEATURES,
1238-
deps = [
1239-
":control_slot_indicator",
1240-
],
1241-
)
1242-
12431198
cc_gtest_unit_test(
12441199
name = "element_fq_id_test",
12451200
srcs = [
@@ -1536,8 +1491,6 @@ cc_unit_test(
15361491
cc_unit_test_suites_for_host_and_qnx(
15371492
name = "unit_test_suite",
15381493
cc_unit_tests = [
1539-
":control_slot_composite_indicator_test",
1540-
":control_slot_indicator_test",
15411494
":element_fq_id_test",
15421495
":event_data_storage_test",
15431496
":event_slot_status_test",

score/mw/com/impl/bindings/lola/control_slot_composite_indicator.cpp

Lines changed: 0 additions & 109 deletions
This file was deleted.

score/mw/com/impl/bindings/lola/control_slot_composite_indicator.h

Lines changed: 0 additions & 75 deletions
This file was deleted.

0 commit comments

Comments
 (0)