Skip to content

Commit ef59501

Browse files
authored
no warnings (#82)
1 parent 6786582 commit ef59501

File tree

4 files changed

+68
-2
lines changed

4 files changed

+68
-2
lines changed

.bazelrc

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,5 +8,7 @@ build:windows --platforms=//bazel/platforms:windows
88
build:linux --platforms=//bazel/platforms:linux
99
build:linux --extra_toolchains=@llvm_toolchain//:cc-toolchain-x86_64-linux
1010

11+
build:no-warnings --per_file_copt=//.*@-Werror
12+
1113
common:ci --announce_rc
1214
common:ci --disk_cache=~/.cache/bazel-disk-cache

.github/workflows/main.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ jobs:
5050
~/.cache/bazel-disk-cache
5151
key: ${{runner.os}}-bazel-cache
5252
- uses: actions/checkout@v3
53-
- run: bazel build --config=ci //...
53+
- run: bazel build --config=ci --config=no-warnings //...
5454
test-linux:
5555
runs-on: ubuntu-latest
5656
needs: build-linux
@@ -62,4 +62,4 @@ jobs:
6262
~/.cache/bazel-disk-cache
6363
key: ${{runner.os}}-bazel-cache
6464
- uses: actions/checkout@v3
65-
- run: bazel test --config=ci //...
65+
- run: bazel test --config=ci --config=no-warnings //...

ecsact/detail/grammar.hh

Lines changed: 51 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -819,4 +819,55 @@ using system_with_entity_level_statement = statement<
819819
system_component_statement,
820820
with_statement>;
821821

822+
constexpr char package_level_statement_name[] = "package level statement";
823+
constexpr char package_level_statement_expected_message[] =
824+
"package level statements are not allowed";
825+
using package_level_statement = statement<
826+
package_level_statement_name,
827+
package_level_statement_expected_message,
828+
none_statement>;
829+
830+
constexpr char import_level_statement_name[] = "import level statement";
831+
constexpr char import_level_statement_expected_message[] =
832+
"import level statements are not allowed";
833+
using import_level_statement = statement<
834+
import_level_statement_name,
835+
import_level_statement_expected_message,
836+
none_statement>;
837+
838+
constexpr char enum_value_level_statement_name[] = "enum value level statement";
839+
constexpr char enum_value_level_statement_expected_message[] =
840+
"enum value level statements are not allowed";
841+
using enum_value_level_statement = statement<
842+
enum_value_level_statement_name,
843+
enum_value_level_statement_expected_message,
844+
none_statement>;
845+
846+
constexpr char builtin_type_field_level_statement_name[] =
847+
"builtin type field level statement";
848+
constexpr char builtin_type_field_level_statement_expected_message[] =
849+
"builtin type field level statements are not allowed";
850+
using builtin_type_field_level_statement = statement<
851+
builtin_type_field_level_statement_name,
852+
builtin_type_field_level_statement_expected_message,
853+
none_statement>;
854+
855+
constexpr char user_type_field_level_statement_name[] =
856+
"user type field level statement";
857+
constexpr char user_type_field_level_statement_expected_message[] =
858+
"user type field level statements are not allowed";
859+
using user_type_field_level_statement = statement<
860+
user_type_field_level_statement_name,
861+
user_type_field_level_statement_expected_message,
862+
none_statement>;
863+
864+
constexpr char entity_constraint_level_statement_name[] =
865+
"entity constraint level statement";
866+
constexpr char entity_constraint_level_statement_expected_message[] =
867+
"entity constraint level statements are not allowed";
868+
using entity_constraint_level_statement = statement<
869+
entity_constraint_level_statement_name,
870+
entity_constraint_level_statement_expected_message,
871+
none_statement>;
872+
822873
} // namespace ecsact::parse::detail::grammar

ecsact/ecsact_parse_statement.cc

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,19 @@ static auto context_grammar(ecsact_statement_type context_type, Fn&& fn) {
3333
return fn(grammar::system_component_level_statement{});
3434
case ECSACT_STATEMENT_SYSTEM_WITH_ENTITY:
3535
return fn(grammar::system_with_entity_level_statement{});
36+
case ECSACT_STATEMENT_PACKAGE:
37+
return fn(grammar::package_level_statement{});
38+
case ECSACT_STATEMENT_IMPORT:
39+
return fn(grammar::import_level_statement{});
40+
case ECSACT_STATEMENT_ENUM_VALUE:
41+
return fn(grammar::enum_value_level_statement{});
42+
case ECSACT_STATEMENT_BUILTIN_TYPE_FIELD:
43+
return fn(grammar::builtin_type_field_level_statement{});
44+
case ECSACT_STATEMENT_USER_TYPE_FIELD:
45+
return fn(grammar::user_type_field_level_statement{});
46+
case ECSACT_STATEMENT_ENTITY_CONSTRAINT:
47+
return fn(grammar::entity_constraint_level_statement{});
48+
break;
3649
}
3750

3851
assert(false && "unhandled context grammar");

0 commit comments

Comments
 (0)