Skip to content

Commit

Permalink
Remove spm-headers ppcdis support
Browse files Browse the repository at this point in the history
  • Loading branch information
SeekyCt committed Sep 3, 2024
1 parent 8757e45 commit db5efed
Show file tree
Hide file tree
Showing 5 changed files with 2 additions and 14 deletions.
2 changes: 0 additions & 2 deletions .vscode.example/c_cpp_properties.json
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,6 @@
],
"defines": [
"DECOMP",
"SKIP_PPCDIS",
"SPM_EU0"
],
"cStandard": "c99",
Expand All @@ -33,7 +32,6 @@
],
"defines": [
"DECOMP",
"SKIP_PPCDIS",
"SPM_EU0"
],
"cStandard": "c99",
Expand Down
1 change: 0 additions & 1 deletion configure.py
Original file line number Diff line number Diff line change
Expand Up @@ -189,7 +189,6 @@

# spm-headers
"-DDECOMP",
"-DSKIP_PPCDIS",
f"-DSPM_{config.version.upper()}",
f"-i {SPM_HEADERS}/include",
f"-i {SPM_HEADERS}/decomp",
Expand Down
4 changes: 0 additions & 4 deletions spm-headers/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -21,10 +21,6 @@ It is recommended to use this with [git subrepo](https://github.com/ingydotnet/g

For use in decomp, the `include` and `decomp` folders should be added to the include path, and the preprocessor define `DECOMP` should be used.

### Testing

The `SKIP_PPCDIS` preprocessor define will stop `ppcdis.h` being included.

## Mods

For use in mods, the `include` and `mod` folder should be added to the include path and an lst from `linker` should be used. If including your compiler's C++ standard library, the preprocessor define `USE_STL` should be used.
Expand Down
2 changes: 1 addition & 1 deletion spm-headers/configure.py
Original file line number Diff line number Diff line change
Expand Up @@ -245,7 +245,7 @@ def test_old_mod_ctx(regions: List[str]):
def test_decomp_ctx(regions: List[str]):
assert args.codewarrior, "Error: decomp_ctx test requires --codewarrior"
compile_regions(os.path.join("$builddir", "{region}", "decomp.o"), "$decomp_source", regions,
DECOMP_INCLUDES, ["DECOMP", "SKIP_PPCDIS"], True)
DECOMP_INCLUDES, ["DECOMP"], True)

# Test shuffled include orders
def test_mod_ctx_shuffle(regions: List[str]):
Expand Down
7 changes: 1 addition & 6 deletions spm-headers/include/common.h
Original file line number Diff line number Diff line change
@@ -1,12 +1,7 @@
#pragma once

#if (defined DECOMP) && !(defined SKIP_PPCDIS)
// Decomp should include ppcdis.h in all files
#include <ppcdis.h>
#else
// Unknown function is useful outside of decomp too
// Unknown function declaration
#define UNKNOWN_FUNCTION(name) void name(void)
#endif

// Intellisense doesn't like asm compiler extensions
#ifdef __INTELLISENSE__
Expand Down

0 comments on commit db5efed

Please sign in to comment.