Skip to content

Commit 095a094

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 151427e commit 095a094

Some content is hidden

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

43 files changed

+485
-1367
lines changed

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

Lines changed: 0 additions & 47 deletions
Original file line numberDiff line numberDiff line change
@@ -561,26 +561,6 @@ cc_library(
561561
],
562562
)
563563

564-
cc_library(
565-
name = "control_slot_indicator",
566-
srcs = [
567-
"control_slot_composite_indicator.cpp",
568-
"control_slot_indicator.cpp",
569-
],
570-
hdrs = [
571-
"control_slot_composite_indicator.h",
572-
"control_slot_indicator.h",
573-
],
574-
features = COMPILER_WARNING_FEATURES,
575-
tags = ["FFI"],
576-
visibility = ["//score/mw/com/impl/bindings/lola:__subpackages__"],
577-
deps = [
578-
":control_slot_types",
579-
":event_slot_status",
580-
"@score_baselibs//score/language/futurecpp",
581-
],
582-
)
583-
584564
cc_library(
585565
name = "event_data_control",
586566
srcs = ["event_data_control.cpp"],
@@ -589,7 +569,6 @@ cc_library(
589569
tags = ["FFI"],
590570
visibility = ["//score/mw/com/impl/bindings/lola:__subpackages__"],
591571
deps = [
592-
":control_slot_indicator",
593572
":control_slot_types",
594573
":transaction_log_set",
595574
"@score_baselibs//score/containers:dynamic_array",
@@ -605,7 +584,6 @@ cc_library(
605584
tags = ["FFI"],
606585
visibility = ["//score/mw/com/impl/bindings/lola:__subpackages__"],
607586
deps = [
608-
":control_slot_indicator",
609587
":control_slot_types",
610588
":event_data_control",
611589
":event_slot_status",
@@ -622,7 +600,6 @@ cc_library(
622600
tags = ["FFI"],
623601
visibility = ["//score/mw/com/impl/bindings/lola:__subpackages__"],
624602
deps = [
625-
":control_slot_indicator",
626603
":control_slot_types",
627604
":event_data_control",
628605
":event_slot_status",
@@ -1232,28 +1209,6 @@ cc_gtest_unit_test(
12321209
],
12331210
)
12341211

1235-
cc_gtest_unit_test(
1236-
name = "control_slot_composite_indicator_test",
1237-
srcs = [
1238-
"control_slot_composite_indicator_test.cpp",
1239-
],
1240-
features = COMPILER_WARNING_FEATURES,
1241-
deps = [
1242-
":control_slot_indicator",
1243-
],
1244-
)
1245-
1246-
cc_gtest_unit_test(
1247-
name = "control_slot_indicator_test",
1248-
srcs = [
1249-
"control_slot_indicator_test.cpp",
1250-
],
1251-
features = COMPILER_WARNING_FEATURES,
1252-
deps = [
1253-
":control_slot_indicator",
1254-
],
1255-
)
1256-
12571212
cc_gtest_unit_test(
12581213
name = "element_fq_id_test",
12591214
srcs = [
@@ -1550,8 +1505,6 @@ cc_unit_test(
15501505
cc_unit_test_suites_for_host_and_qnx(
15511506
name = "unit_test_suite",
15521507
cc_unit_tests = [
1553-
":control_slot_composite_indicator_test",
1554-
":control_slot_indicator_test",
15551508
":element_fq_id_test",
15561509
":event_data_storage_test",
15571510
":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)