diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml index 3467d692..2ca671fd 100644 --- a/.github/workflows/build.yml +++ b/.github/workflows/build.yml @@ -17,7 +17,7 @@ jobs: os_identifier: "win" os: windows-2019 prefixes: '' - extra_flags: '-G "Visual Studio 16 2019" -T host=x86 -A win32 -DGEODE_DEBUG=On' + extra_flags: '-T host=x86 -A win32 -DGEODE_DEBUG=On' out_paths: './bin/nightly/Geode.dll ./bin/nightly/GeodeBootstrapper.dll ./bin/nightly/Geode.lib ./bin/nightly/XInput9_1_0.dll' cli_cmd: '' @@ -141,5 +141,6 @@ jobs: with: token: ${{ secrets.GITHUB_TOKEN }} files: ./geode-nightly-mac.zip ./geode-nightly-win.zip - release: Nightly + release: nightly + tag: nightly prerelease: true diff --git a/CHANGELOG.md b/CHANGELOG.md index cf26c84f..879594fb 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -1,9 +1,16 @@ # Geode Changelog +## v1.0.0-beta.9 + * Fix multiple modifiers not being able to have fields on same class due to having same field index (7710fa9) + * Add `Result::ok` and `Result::err` for converting the `Result` into `std::optional` (4a15afc) + ## v1.0.0-beta.8 * Unload the mod even when first time warning pops up (63b4774) - * Make log not throw, but warn on invalid format (6aba7cf) * Error when address of a function returns nullptr when hooking (724a9d3) + * Add support for Geode CLI v2.0.0 (which has not been released yet) (088ac7b, deadb58) + * Logging no longer causes a crash on invalid formats, but instead just warns (6aba7cf) + * `file::pickFile` now uses the last item in the default path as the default filename to save/open (5c9ee08) + * Fix `EditorPauseLayer` crashing constantly due to some members being accidentally set to `nullptr` (33a91d6) ## v1.0.0-beta.7 * Add `Mod::getResourcesDir` for getting the mod resources directory (0055032) diff --git a/bindings/GeometryDash.bro b/bindings/GeometryDash.bro index 6fb6f127..b4d93caf 100644 --- a/bindings/GeometryDash.bro +++ b/bindings/GeometryDash.bro @@ -866,7 +866,7 @@ class CreatorLayer : cocos2d::CCLayer { void onFameLevels(cocos2d::CCObject*) = win 0x4ee70; void onMapPacks(cocos2d::CCObject*) = win 0x4efb0; void onOnlineLevels(cocos2d::CCObject*) = win 0x4ef60; - void onGauntlets(cocos2d::CCObject*) = win 0x4f0a0; + void onGauntlets(cocos2d::CCObject*) = mac 0x142b20, win 0x4f0a0; void onSecretVault(cocos2d::CCObject*) = win 0x4f1d0; void onTreasureRoom(cocos2d::CCObject*) = win 0x4f540; virtual void sceneWillResume() = win 0x4fb50; @@ -878,6 +878,13 @@ class CurrencyRewardLayer : cocos2d::CCLayer { inline CurrencyRewardLayer() {} ~CurrencyRewardLayer() = mac 0x447950, win 0x4ffb0; virtual void update(float) = mac 0x44a5c0, win 0x52350; + + static CurrencyRewardLayer* create( + int, int, int, + CurrencySpriteType, int, + CurrencySpriteType, int, + cocos2d::CCPoint, CurrencyRewardType, float + ) = win 0x50050; } class CurrencyRewardDelegate { @@ -3476,7 +3483,7 @@ class LeaderboardManagerDelegate {} class LeaderboardsLayer : cocos2d::CCLayer { static LeaderboardsLayer* create(LeaderboardState state) = win 0x158710; - bool init(LeaderboardState state) = win 0x1587b0; + bool init(LeaderboardState state) = mac 0x29f6d0, win 0x1587b0; } class LevelBrowserLayer : cocos2d::CCLayer { @@ -3763,7 +3770,7 @@ class LevelInfoLayer : cocos2d::CCLayer, LevelDownloadDelegate, LevelUpdateDeleg class LevelLeaderboard : FLAlertLayer { void onChangeType(cocos2d::CCObject* sender) = win 0x17d090; void onGarage(cocos2d::CCObject* sender) = win 0x17d1b0; - bool init(GJGameLevel* level, int type) = win 0x17c4f0; + bool init(GJGameLevel* level, int type) = mac 0x20d710, win 0x17c4f0; static LevelLeaderboard* create(GJGameLevel* level, LevelLeaderboardType leaderboardType) = win 0x17c440; } @@ -3929,6 +3936,7 @@ class LocalLevelManager : cocos2d::CCNode { inline static LocalLevelManager* get() { return LocalLevelManager::sharedState(); } + bool init() = mac 0x2384e0; PAD = mac 0x10, win 0x1C; cocos2d::CCDictionary* m_loadData; @@ -4284,7 +4292,7 @@ class PlayLayer : GJBaseGameLayer, CCCircleWaveDelegate, CurrencyRewardDelegate, void shouldBlend(int) = mac 0x771b0; void showCompleteEffect() = mac 0x738e0, win 0x1fe060; void showCompleteText() = mac 0x73be0, win 0x1fda90; - void showEndLayer() = mac 0x74450; + void showEndLayer() = mac 0x74450, win 0x1ffd00; void showHint() = mac 0x7deb0; void showNewBest(bool, int, int, bool, bool, bool) = mac 0x74580, win 0x1fe3a0; void showRetryLayer() = mac 0x75ba0; @@ -5406,11 +5414,15 @@ class VideoOptionsLayer : FLAlertLayer { } class LevelTools { - static gd::string base64EncodeString(gd::string) = win 0x18b310; + static gd::string base64EncodeString(gd::string) = mac 0x294470, win 0x18b310; static gd::string base64DecodeString(gd::string) = mac 0x294510, win 0x18b3b0; - static GJGameLevel *getLevel(int, bool) = win 0x189370; - static bool verifyLevelIntegrity(gd::string, int) = win 0x18b180; - static float xPosForTime(float, cocos2d::CCArray*, int) = win 0x18acd0; - static float timeForXPos(float, cocos2d::CCArray*, int) = win 0x18ae70; + static GJGameLevel* getLevel(int, bool) = mac 0x2908c0, win 0x189370; + static bool verifyLevelIntegrity(gd::string, int) = mac 0x294360, win 0x18b180; + static float xPosForTime(float, cocos2d::CCArray*, int) = mac 0x293d90, win 0x18acd0; + static float timeForXPos(float, cocos2d::CCArray*, int) = mac 0x293eb0, win 0x18ae70; + static gd::string getAudioFilename(int) = mac 0x292840; + static gd::string getAudioTitle(int) = mac 0x2922f0; + static gd::string getArtistForAudio(int) = mac 0x292d90; + static gd::string getURLForAudio(int) = mac 0x292f10; } // clang-format on diff --git a/codegen/src/BindingGen.cpp b/codegen/src/BindingGen.cpp index c87fb974..17f8de11 100644 --- a/codegen/src/BindingGen.cpp +++ b/codegen/src/BindingGen.cpp @@ -34,10 +34,10 @@ public: static constexpr auto CLASS_NAME = "{class_name}"; )GEN"; - char const* monostate_constructor = R"GEN( GEODE_MONOSTATE_CONSTRUCTOR_GD({class_name}, {first_base}) + char const* custom_constructor = R"GEN( GEODE_CUSTOM_CONSTRUCTOR_GD({class_name}, {first_base}) )GEN"; - char const* monostate_constructor_cutoff = R"GEN( GEODE_MONOSTATE_CONSTRUCTOR_CUTOFF({class_name}, {first_base}) + char const* custom_constructor_cutoff = R"GEN( GEODE_CUSTOM_CONSTRUCTOR_CUTOFF({class_name}, {first_base}) )GEN"; char const* function_definition = R"GEN( @@ -181,8 +181,8 @@ std::string generateBindingHeader(Root& root, ghc::filesystem::path const& singl if (!cls.superclasses.empty()) { single_output += fmt::format( can_find(cls.superclasses[0], "cocos2d") - ? format_strings::monostate_constructor_cutoff - : format_strings::monostate_constructor, + ? format_strings::custom_constructor_cutoff + : format_strings::custom_constructor, fmt::arg("class_name", cls.name), fmt::arg("first_base", cls.superclasses[0]) ); diff --git a/codegen/src/SourceGen.cpp b/codegen/src/SourceGen.cpp index 0ce659c8..83b3f380 100644 --- a/codegen/src/SourceGen.cpp +++ b/codegen/src/SourceGen.cpp @@ -85,13 +85,13 @@ auto {class_name}::{function_name}({parameters}){const} -> decltype({function_na reinterpret_cast(func)(this{parameter_comma}{arguments}); // we need to construct it back so that it uhhh ummm doesnt crash // while going to the child destructors - auto thing = new (this) {class_name}(std::monostate(), sizeof({class_name})); + auto thing = new (this) {class_name}(geode::CutoffConstructor, sizeof({class_name})); CCDestructor::lock(this) = true; }} )GEN"; char const* declare_constructor = R"GEN( -{class_name}::{function_name}({parameters}) : {class_name}(std::monostate(), sizeof({class_name})) {{ +{class_name}::{function_name}({parameters}) : {class_name}(geode::CutoffConstructor, sizeof({class_name})) {{ // here we construct it as normal as we can, then destruct it // using the generated functions. this ensures no memory gets leaked // no crashes :pray: diff --git a/loader/include/Geode/DefaultInclude.hpp b/loader/include/Geode/DefaultInclude.hpp index 90c48f47..6956f302 100644 --- a/loader/include/Geode/DefaultInclude.hpp +++ b/loader/include/Geode/DefaultInclude.hpp @@ -69,28 +69,74 @@ namespace cocos2d::extension {} #define GEODE_EXPAND(x) x #define GEODE_INVOKE(macro, ...) GEODE_EXPAND(macro(__VA_ARGS__)) -#define GEODE_FILL_CONSTRUCTOR(Class_, Offset_) \ - Class_(std::monostate, size_t fill) : \ - Class_({}, std::memset(reinterpret_cast(this) + Offset_, 0, fill - Offset_)) {} \ - Class_(std::monostate, void*) +namespace geode { + struct ZeroConstructorType {}; -#define GEODE_MONOSTATE_CONSTRUCTOR_BEGIN(Class_) \ + static constexpr auto ZeroConstructor = ZeroConstructorType(); + + struct CutoffConstructorType {}; + + static constexpr auto CutoffConstructor = CutoffConstructorType(); +} + +#define GEODE_CUSTOM_CONSTRUCTOR_BEGIN(Class_) \ + GEODE_ZERO_CONSTRUCTOR_BEGIN(Class_) \ + GEODE_CUTOFF_CONSTRUCTOR_BEGIN(Class_) + +#define GEODE_CUSTOM_CONSTRUCTOR_COCOS(Class_, Base_) \ + GEODE_ZERO_CONSTRUCTOR(Class_, Base_) \ + GEODE_CUTOFF_CONSTRUCTOR_COCOS(Class_, Base_) + +#define GEODE_CUSTOM_CONSTRUCTOR_GD(Class_, Base_) \ + GEODE_ZERO_CONSTRUCTOR(Class_, Base_) \ + GEODE_CUTOFF_CONSTRUCTOR_GD(Class_, Base_) + +#define GEODE_CUSTOM_CONSTRUCTOR_CUTOFF(Class_, Base_) \ + GEODE_ZERO_CONSTRUCTOR(Class_, Base_) \ + GEODE_CUTOFF_CONSTRUCTOR_CUTOFF(Class_, Base_) + +#define GEODE_ZERO_CONSTRUCTOR_BEGIN(Class_) \ + Class_(geode::ZeroConstructorType, void*) {} \ + Class_(geode::ZeroConstructorType, size_t fill) : \ + Class_(geode::ZeroConstructor, std::memset(static_cast(this), 0, fill)) {} \ + Class_(geode::ZeroConstructorType) : Class_(geode::ZeroConstructor, nullptr) {} + +#define GEODE_ZERO_CONSTRUCTOR(Class_, Base_) \ + Class_(geode::ZeroConstructorType, size_t fill) : Base_(geode::ZeroConstructor, fill) {} \ + Class_(geode::ZeroConstructorType) : Base_(geode::ZeroConstructor, sizeof(Class_)) {} + +#define GEODE_FILL_CONSTRUCTOR(Class_, Offset_) \ + Class_(geode::CutoffConstructorType, size_t fill) : \ + Class_( \ + geode::CutoffConstructor, \ + std::memset(reinterpret_cast(this) + Offset_, 0, fill - Offset_) \ + ) {} \ + Class_(geode::CutoffConstructorType, void*) + +#define GEODE_CUTOFF_CONSTRUCTOR_BEGIN(Class_) \ GEODE_MACOS(GEODE_FILL_CONSTRUCTOR(Class_, 0){}) \ GEODE_IOS(GEODE_FILL_CONSTRUCTOR(Class_, 0){}) -#define GEODE_MONOSTATE_CONSTRUCTOR_COCOS(Class_, Base_) \ - GEODE_MACOS(Class_(std::monostate, size_t fill) : Base_({}, fill){}) \ - GEODE_IOS(Class_(std::monostate, size_t fill) : Base_({}, fill){}) +#define GEODE_CUTOFF_CONSTRUCTOR_COCOS(Class_, Base_) \ + GEODE_MACOS(Class_(geode::CutoffConstructorType, size_t fill) \ + : Base_(geode::CutoffConstructor, fill){}) \ + GEODE_IOS(Class_(geode::CutoffConstructorType, size_t fill) \ + : Base_(geode::CutoffConstructor, fill){}) -#define GEODE_MONOSTATE_CONSTRUCTOR_GD(Class_, Base_) \ - GEODE_WINDOWS(Class_(std::monostate, size_t fill) : Base_({}, fill){}) \ - GEODE_MACOS(Class_(std::monostate, size_t fill) : Base_({}, fill){}) \ - GEODE_IOS(Class_(std::monostate, size_t fill) : Base_({}, fill){}) +#define GEODE_CUTOFF_CONSTRUCTOR_GD(Class_, Base_) \ + GEODE_WINDOWS(Class_(geode::CutoffConstructorType, size_t fill) \ + : Base_(geode::CutoffConstructor, fill){}) \ + GEODE_MACOS(Class_(geode::CutoffConstructorType, size_t fill) \ + : Base_(geode::CutoffConstructor, fill){}) \ + GEODE_IOS(Class_(geode::CutoffConstructorType, size_t fill) \ + : Base_(geode::CutoffConstructor, fill){}) -#define GEODE_MONOSTATE_CONSTRUCTOR_CUTOFF(Class_, Base_) \ +#define GEODE_CUTOFF_CONSTRUCTOR_CUTOFF(Class_, Base_) \ GEODE_WINDOWS(GEODE_FILL_CONSTRUCTOR(Class_, sizeof(Base_)) : Base_(){}) \ - GEODE_MACOS(Class_(std::monostate, size_t fill) : Base_({}, fill){}) \ - GEODE_IOS(Class_(std::monostate, size_t fill) : Base_({}, fill){}) + GEODE_MACOS(Class_(geode::CutoffConstructorType, size_t fill) \ + : Base_(geode::CutoffConstructor, fill){}) \ + GEODE_IOS(Class_(geode::CutoffConstructorType, size_t fill) \ + : Base_(geode::CutoffConstructor, fill){}) #define GEODE_NUMBER_OF_ARGS(...) \ GEODE_EXPAND(GEODE_NUMBER_OF_ARGS_(__VA_ARGS__, GEODE_NUMBER_SEQUENCE(), )) diff --git a/loader/include/Geode/Enums.hpp b/loader/include/Geode/Enums.hpp index 42eb3bc3..f7a052d5 100644 --- a/loader/include/Geode/Enums.hpp +++ b/loader/include/Geode/Enums.hpp @@ -122,6 +122,14 @@ enum class BoomListType { Artist = 0x14, }; +enum class CurrencySpriteType { + // todo +}; + +enum class CurrencyRewardType { + // todo +}; + enum class MenuAnimationType { Scale = 0, Move = 1, diff --git a/loader/include/Geode/cocos/CCDirector.h b/loader/include/Geode/cocos/CCDirector.h index 093415ed..385a137a 100644 --- a/loader/include/Geode/cocos/CCDirector.h +++ b/loader/include/Geode/cocos/CCDirector.h @@ -121,6 +121,7 @@ public: * @js ctor */ CCDirector(void); + GEODE_CUSTOM_CONSTRUCTOR_COCOS(CCDirector, CCObject) /** * @js NA * @lua NA @@ -146,7 +147,7 @@ public: /** Get the FPS value */ inline double getAnimationInterval(void) { return m_dAnimationInterval; } /** Set the FPS value. */ - virtual void setAnimationInterval(double dValue) = 0; + virtual void setAnimationInterval(double dValue) {} /** Whether or not to display the FPS on the bottom-left corner */ inline bool isDisplayStats(void) { return m_bDisplayStats; } @@ -308,13 +309,13 @@ protected: /** Stops the animation. Nothing will be drawn. The main loop won't be triggered anymore. If you don't want to pause your animation call [pause] instead. */ - virtual void stopAnimation(void) = 0; + virtual void stopAnimation(void) {} /** The main loop is triggered again. Call this function only if [stopAnimation] was called earlier @warning Don't call this function to start the main loop. To run the main loop call runWithScene */ - virtual void startAnimation(void) = 0; + virtual void startAnimation(void) {} public: /** Draw the scene. @@ -345,7 +346,7 @@ public: void setDepthTest(bool bOn); protected: - virtual void mainLoop(void) = 0; + virtual void mainLoop(void) {} /** The size in pixels of the surface. It could be different than the screen size. High-res devices might have a higher surface size than the screen size. diff --git a/loader/include/Geode/cocos/CCScheduler.h b/loader/include/Geode/cocos/CCScheduler.h index bbfdb9bc..59547521 100644 --- a/loader/include/Geode/cocos/CCScheduler.h +++ b/loader/include/Geode/cocos/CCScheduler.h @@ -141,6 +141,7 @@ class CC_DLL CCScheduler : public CCObject GEODE_FRIEND_MODIFY public: CCScheduler(); + GEODE_CUSTOM_CONSTRUCTOR_COCOS(CCScheduler, CCObject) /** * @js NA * @lua NA diff --git a/loader/include/Geode/cocos/base_nodes/CCNode.h b/loader/include/Geode/cocos/base_nodes/CCNode.h index 809f4e97..eb73462f 100644 --- a/loader/include/Geode/cocos/base_nodes/CCNode.h +++ b/loader/include/Geode/cocos/base_nodes/CCNode.h @@ -140,7 +140,7 @@ public: * @js ctor */ CCNode(void); - GEODE_MONOSTATE_CONSTRUCTOR_COCOS(CCNode, CCObject) + GEODE_CUSTOM_CONSTRUCTOR_COCOS(CCNode, CCObject) /** * Default destructor @@ -1638,7 +1638,7 @@ public: * @js ctor */ CCNodeRGBA(); - GEODE_MONOSTATE_CONSTRUCTOR_COCOS(CCNodeRGBA, CCNode) + GEODE_CUSTOM_CONSTRUCTOR_COCOS(CCNodeRGBA, CCNode) /** * @js NA * @lua NA diff --git a/loader/include/Geode/cocos/base_nodes/Layout.hpp b/loader/include/Geode/cocos/base_nodes/Layout.hpp index afc1d19b..bd4e23ac 100644 --- a/loader/include/Geode/cocos/base_nodes/Layout.hpp +++ b/loader/include/Geode/cocos/base_nodes/Layout.hpp @@ -234,7 +234,8 @@ protected: CCNode* on, CCArray* nodes, std::pair const& minMaxPrios, bool doAutoScale, - float scale, float squish, int prio + float scale, float squish, int prio, + size_t depth ) const; AxisLayout(Axis); diff --git a/loader/include/Geode/cocos/cocoa/CCObject.h b/loader/include/Geode/cocos/cocoa/CCObject.h index 1f7adfae..daeb19bf 100644 --- a/loader/include/Geode/cocos/cocoa/CCObject.h +++ b/loader/include/Geode/cocos/cocoa/CCObject.h @@ -112,7 +112,7 @@ protected: int m_nUnknown; ) public: - GEODE_MONOSTATE_CONSTRUCTOR_BEGIN(CCObject) + GEODE_CUSTOM_CONSTRUCTOR_BEGIN(CCObject) CCObject(void); /** * @lua NA diff --git a/loader/include/Geode/cocos/cocoa/CCSet.h b/loader/include/Geode/cocos/cocoa/CCSet.h index 49784576..4ec1c0bf 100644 --- a/loader/include/Geode/cocos/cocoa/CCSet.h +++ b/loader/include/Geode/cocos/cocoa/CCSet.h @@ -46,7 +46,7 @@ public: * @lua NA */ CCSet(void); - GEODE_MONOSTATE_CONSTRUCTOR_COCOS(CCSet, CCObject) + GEODE_CUSTOM_CONSTRUCTOR_COCOS(CCSet, CCObject) /** * @lua NA */ diff --git a/loader/include/Geode/cocos/cocoa/CCString.h b/loader/include/Geode/cocos/cocoa/CCString.h index 21bc709f..713aa0d0 100644 --- a/loader/include/Geode/cocos/cocoa/CCString.h +++ b/loader/include/Geode/cocos/cocoa/CCString.h @@ -47,6 +47,7 @@ class CC_DLL CCString : public CCObject { GEODE_FRIEND_MODIFY public: + GEODE_CUSTOM_CONSTRUCTOR_COCOS(CCString, CCObject) /** * @lua NA */ diff --git a/loader/include/Geode/cocos/draw_nodes/CCDrawNode.h b/loader/include/Geode/cocos/draw_nodes/CCDrawNode.h index e6659267..495c52d0 100644 --- a/loader/include/Geode/cocos/draw_nodes/CCDrawNode.h +++ b/loader/include/Geode/cocos/draw_nodes/CCDrawNode.h @@ -94,7 +94,7 @@ public: void setBlendFunc(const ccBlendFunc &blendFunc); CCDrawNode(); - GEODE_MONOSTATE_CONSTRUCTOR_COCOS(CCDrawNode, CCNodeRGBA) + GEODE_CUSTOM_CONSTRUCTOR_COCOS(CCDrawNode, CCNodeRGBA) /** listen the event that coming to foreground on Android * @js NA diff --git a/loader/include/Geode/cocos/extensions/GUI/CCControlExtension/CCControl.h b/loader/include/Geode/cocos/extensions/GUI/CCControlExtension/CCControl.h index 3ad32bc7..1db66ae5 100644 --- a/loader/include/Geode/cocos/extensions/GUI/CCControlExtension/CCControl.h +++ b/loader/include/Geode/cocos/extensions/GUI/CCControlExtension/CCControl.h @@ -139,7 +139,7 @@ public: * @js ctor */ CCControl(); - GEODE_MONOSTATE_CONSTRUCTOR_COCOS(CCControl, CCLayerRGBA) + GEODE_CUSTOM_CONSTRUCTOR_COCOS(CCControl, CCLayerRGBA) virtual bool init(void); /** diff --git a/loader/include/Geode/cocos/extensions/GUI/CCControlExtension/CCControlColourPicker.h b/loader/include/Geode/cocos/extensions/GUI/CCControlExtension/CCControlColourPicker.h index d1c6fa25..2c0a7418 100644 --- a/loader/include/Geode/cocos/extensions/GUI/CCControlExtension/CCControlColourPicker.h +++ b/loader/include/Geode/cocos/extensions/GUI/CCControlExtension/CCControlColourPicker.h @@ -58,7 +58,7 @@ public: virtual void setColorValue(ccColor3B const&); ) - GEODE_MONOSTATE_CONSTRUCTOR_COCOS(CCControlColourPicker, CCControl) + GEODE_CUSTOM_CONSTRUCTOR_COCOS(CCControlColourPicker, CCControl) CCControlColourPicker(); virtual ~CCControlColourPicker(); diff --git a/loader/include/Geode/cocos/extensions/GUI/CCControlExtension/CCScale9Sprite.h b/loader/include/Geode/cocos/extensions/GUI/CCControlExtension/CCScale9Sprite.h index d3176597..43d9c136 100644 --- a/loader/include/Geode/cocos/extensions/GUI/CCControlExtension/CCScale9Sprite.h +++ b/loader/include/Geode/cocos/extensions/GUI/CCControlExtension/CCScale9Sprite.h @@ -54,7 +54,7 @@ class CC_DLL CCScale9Sprite : public CCNodeRGBA { public: CCScale9Sprite(); - GEODE_MONOSTATE_CONSTRUCTOR_COCOS(CCScale9Sprite, CCNodeRGBA) + GEODE_CUSTOM_CONSTRUCTOR_COCOS(CCScale9Sprite, CCNodeRGBA) virtual ~CCScale9Sprite(); public: diff --git a/loader/include/Geode/cocos/extensions/GUI/CCScrollView/CCScrollView.h b/loader/include/Geode/cocos/extensions/GUI/CCScrollView/CCScrollView.h index 3e5bfeef..30918b1d 100644 --- a/loader/include/Geode/cocos/extensions/GUI/CCScrollView/CCScrollView.h +++ b/loader/include/Geode/cocos/extensions/GUI/CCScrollView/CCScrollView.h @@ -69,7 +69,7 @@ public: * @js ctor */ CCScrollView(); - GEODE_MONOSTATE_CONSTRUCTOR_COCOS(CCScrollView, CCLayer) + GEODE_CUSTOM_CONSTRUCTOR_COCOS(CCScrollView, CCLayer) /** * @js NA * @lua NA diff --git a/loader/include/Geode/cocos/keypad_dispatcher/CCKeypadDelegate.h b/loader/include/Geode/cocos/keypad_dispatcher/CCKeypadDelegate.h index cf588eba..987c2f99 100644 --- a/loader/include/Geode/cocos/keypad_dispatcher/CCKeypadDelegate.h +++ b/loader/include/Geode/cocos/keypad_dispatcher/CCKeypadDelegate.h @@ -59,7 +59,7 @@ class CC_DLL CCKeypadHandler : public CCObject { GEODE_FRIEND_MODIFY public: - GEODE_MONOSTATE_CONSTRUCTOR_COCOS(CCKeypadHandler, CCObject) + GEODE_CUSTOM_CONSTRUCTOR_COCOS(CCKeypadHandler, CCObject) inline CCKeypadHandler() = default; virtual ~CCKeypadHandler(void); diff --git a/loader/include/Geode/cocos/label_nodes/CCLabelBMFont.h b/loader/include/Geode/cocos/label_nodes/CCLabelBMFont.h index d7a90866..1fa7e2ac 100644 --- a/loader/include/Geode/cocos/label_nodes/CCLabelBMFont.h +++ b/loader/include/Geode/cocos/label_nodes/CCLabelBMFont.h @@ -198,7 +198,7 @@ public: * @js ctor */ CCLabelBMFont(); - GEODE_MONOSTATE_CONSTRUCTOR_COCOS(CCLabelBMFont, CCSpriteBatchNode) + GEODE_CUSTOM_CONSTRUCTOR_COCOS(CCLabelBMFont, CCSpriteBatchNode) /** * @js NA * @lua NA diff --git a/loader/include/Geode/cocos/label_nodes/CCLabelTTF.h b/loader/include/Geode/cocos/label_nodes/CCLabelTTF.h index 721d8b60..6bb86c77 100644 --- a/loader/include/Geode/cocos/label_nodes/CCLabelTTF.h +++ b/loader/include/Geode/cocos/label_nodes/CCLabelTTF.h @@ -63,6 +63,7 @@ public: * @js ctor */ CCLabelTTF(); + GEODE_CUSTOM_CONSTRUCTOR_COCOS(CCLabelTTF, CCSprite) /** * @js NA * @lua NA diff --git a/loader/include/Geode/cocos/layers_scenes_transitions_nodes/CCLayer.h b/loader/include/Geode/cocos/layers_scenes_transitions_nodes/CCLayer.h index 512fdf6a..7ea96fd0 100644 --- a/loader/include/Geode/cocos/layers_scenes_transitions_nodes/CCLayer.h +++ b/loader/include/Geode/cocos/layers_scenes_transitions_nodes/CCLayer.h @@ -75,7 +75,7 @@ public: * @js ctor */ CCLayer(); - GEODE_MONOSTATE_CONSTRUCTOR_COCOS(CCLayer, CCNode) + GEODE_CUSTOM_CONSTRUCTOR_COCOS(CCLayer, CCNode) /** * @js NA * @lua NA @@ -233,7 +233,7 @@ public: * @js ctor */ CCLayerRGBA(); - GEODE_MONOSTATE_CONSTRUCTOR_COCOS(CCLayerRGBA, CCLayer) + GEODE_CUSTOM_CONSTRUCTOR_COCOS(CCLayerRGBA, CCLayer) /** * @js NA * @lua NA @@ -290,7 +290,7 @@ public: * @js ctor */ CCLayerColor(); - GEODE_MONOSTATE_CONSTRUCTOR_COCOS(CCLayerColor, CCLayerRGBA) + GEODE_CUSTOM_CONSTRUCTOR_COCOS(CCLayerColor, CCLayerRGBA) /** * @js NA * @lua NA diff --git a/loader/include/Geode/cocos/layers_scenes_transitions_nodes/CCScene.h b/loader/include/Geode/cocos/layers_scenes_transitions_nodes/CCScene.h index 21003355..433cb390 100644 --- a/loader/include/Geode/cocos/layers_scenes_transitions_nodes/CCScene.h +++ b/loader/include/Geode/cocos/layers_scenes_transitions_nodes/CCScene.h @@ -63,7 +63,7 @@ public: * @js ctor */ CCScene(); - GEODE_MONOSTATE_CONSTRUCTOR_COCOS(CCScene, CCNode) + GEODE_CUSTOM_CONSTRUCTOR_COCOS(CCScene, CCNode) /** * @js NA * @lua NA diff --git a/loader/include/Geode/cocos/menu_nodes/CCMenu.h b/loader/include/Geode/cocos/menu_nodes/CCMenu.h index dedd67f0..faecdc79 100644 --- a/loader/include/Geode/cocos/menu_nodes/CCMenu.h +++ b/loader/include/Geode/cocos/menu_nodes/CCMenu.h @@ -64,7 +64,7 @@ public: * @js ctor */ CCMenu() : m_pSelectedItem(NULL) {} - GEODE_MONOSTATE_CONSTRUCTOR_COCOS(CCMenu, CCLayerRGBA) + GEODE_CUSTOM_CONSTRUCTOR_COCOS(CCMenu, CCLayerRGBA) /** * @js NA * @lua NA diff --git a/loader/include/Geode/cocos/menu_nodes/CCMenuItem.h b/loader/include/Geode/cocos/menu_nodes/CCMenuItem.h index 4a44c453..83761988 100644 --- a/loader/include/Geode/cocos/menu_nodes/CCMenuItem.h +++ b/loader/include/Geode/cocos/menu_nodes/CCMenuItem.h @@ -71,7 +71,7 @@ public: , m_pfnSelector(NULL) , m_nScriptTapHandler(0) {} - GEODE_MONOSTATE_CONSTRUCTOR_COCOS(CCMenuItem, CCNodeRGBA) + GEODE_CUSTOM_CONSTRUCTOR_COCOS(CCMenuItem, CCNodeRGBA) /** * @js NA * @lua NA @@ -139,7 +139,7 @@ public: : m_pLabel(NULL) , m_fOriginalScale(0.0) {} - GEODE_MONOSTATE_CONSTRUCTOR_COCOS(CCMenuItemLabel, CCMenuItem) + GEODE_CUSTOM_CONSTRUCTOR_COCOS(CCMenuItemLabel, CCMenuItem) /** * @js NA * @lua NA @@ -292,7 +292,7 @@ public: ,m_pSelectedImage(NULL) ,m_pDisabledImage(NULL) {} - GEODE_MONOSTATE_CONSTRUCTOR_COCOS(CCMenuItemSprite, CCMenuItem) + GEODE_CUSTOM_CONSTRUCTOR_COCOS(CCMenuItemSprite, CCMenuItem) /** creates a menu item with a normal, selected and disabled image*/ static CCMenuItemSprite * create(CCNode* normalSprite, CCNode* selectedSprite, CCNode* disabledSprite = NULL); @@ -337,7 +337,7 @@ public: * @lua NA */ CCMenuItemImage(){} - GEODE_MONOSTATE_CONSTRUCTOR_COCOS(CCMenuItemImage, CCMenuItemSprite) + GEODE_CUSTOM_CONSTRUCTOR_COCOS(CCMenuItemImage, CCMenuItemSprite) /** * @js NA * @lua NA diff --git a/loader/include/Geode/cocos/platform/CCApplicationProtocol.h b/loader/include/Geode/cocos/platform/CCApplicationProtocol.h index 1acca89e..4aeb1001 100644 --- a/loader/include/Geode/cocos/platform/CCApplicationProtocol.h +++ b/loader/include/Geode/cocos/platform/CCApplicationProtocol.h @@ -62,24 +62,24 @@ public: @return true Initialize success, app continue. @return false Initialize failed, app terminate. */ - virtual bool applicationDidFinishLaunching() = 0; + virtual bool applicationDidFinishLaunching() { return true; } /** @brief The function be called when the application enter background @param the pointer of the application */ - virtual void applicationDidEnterBackground() = 0; + virtual void applicationDidEnterBackground() {} /** @brief The function be called when the application enter foreground @param the pointer of the application */ - virtual void applicationWillEnterForeground() = 0; + virtual void applicationWillEnterForeground() {} RT_ADD( virtual void applicationWillBecomeActive() {} virtual void applicationWillResignActive() {} - virtual void trySaveGame() = 0; + virtual void trySaveGame() {} virtual void gameDidSave() {} ) @@ -87,18 +87,18 @@ public: @brief Callback by CCDirector for limit FPS. @interval The time, expressed in seconds, between current frame and next. */ - virtual void setAnimationInterval(double interval) = 0; + virtual void setAnimationInterval(double interval) {} /** @brief Get current language config @return Current language config */ - virtual ccLanguageType getCurrentLanguage() = 0; + virtual ccLanguageType getCurrentLanguage() { return kLanguageEnglish; } /** @brief Get target platform */ - virtual TargetPlatform getTargetPlatform() = 0; + virtual TargetPlatform getTargetPlatform() { return kTargetWindows; } RT_ADD( virtual void openURL(const char* url) {} ) }; diff --git a/loader/include/Geode/cocos/platform/CCFileUtils.h b/loader/include/Geode/cocos/platform/CCFileUtils.h index 98341239..0df6c7fc 100644 --- a/loader/include/Geode/cocos/platform/CCFileUtils.h +++ b/loader/include/Geode/cocos/platform/CCFileUtils.h @@ -41,8 +41,8 @@ class CCArray; */ struct CCTexturePack { - std::string m_id; - std::vector m_paths; + std::string m_id; + std::vector m_paths; }; //! @brief Helper class to handle file operations @@ -52,6 +52,8 @@ class CC_DLL CCFileUtils : public TypeInfo friend class CCArray; friend class CCDictionary; public: + + GEODE_CUSTOM_CONSTRUCTOR_BEGIN(CCFileUtils) /** * Returns an unique ID for this class. * @note It's only used for JSBindings now. @@ -339,7 +341,7 @@ public: * @return The path that can be write/read a file in * @lua NA */ - virtual gd::string getWritablePath() = 0; + virtual gd::string getWritablePath() { return ""; } RT_ADD(virtual gd::string getWritablePath2();) @@ -351,7 +353,7 @@ public: * @return true if the file exists, otherwise it will return false. * @lua NA */ - virtual bool isFileExist(const gd::string& strFilePath) = 0; + virtual bool isFileExist(const gd::string& strFilePath) { return false; } /** * Checks whether the path is an absolute path. diff --git a/loader/include/Geode/cocos/platform/CCImage.h b/loader/include/Geode/cocos/platform/CCImage.h index 131cfe1b..a71af4e8 100644 --- a/loader/include/Geode/cocos/platform/CCImage.h +++ b/loader/include/Geode/cocos/platform/CCImage.h @@ -46,7 +46,7 @@ public: @js ctor */ CCImage(); - GEODE_MONOSTATE_CONSTRUCTOR_COCOS(CCImage, CCObject) + GEODE_CUSTOM_CONSTRUCTOR_COCOS(CCImage, CCObject) /** * @js NA * @lua NA diff --git a/loader/include/Geode/cocos/platform/ios/CCApplication.h b/loader/include/Geode/cocos/platform/ios/CCApplication.h index 85467639..76b17efd 100644 --- a/loader/include/Geode/cocos/platform/ios/CCApplication.h +++ b/loader/include/Geode/cocos/platform/ios/CCApplication.h @@ -36,7 +36,7 @@ class CC_DLL CCApplication : public CCApplicationProtocol { GEODE_FRIEND_MODIFY public: - GEODE_MONOSTATE_CONSTRUCTOR_BEGIN(CCApplication) + GEODE_CUSTOM_CONSTRUCTOR_BEGIN(CCApplication) CCApplication(); /** * @js NA diff --git a/loader/include/Geode/cocos/platform/mac/CCApplication.h b/loader/include/Geode/cocos/platform/mac/CCApplication.h index 57562274..79c3fc88 100644 --- a/loader/include/Geode/cocos/platform/mac/CCApplication.h +++ b/loader/include/Geode/cocos/platform/mac/CCApplication.h @@ -35,7 +35,7 @@ class CC_DLL CCApplication : public CCApplicationProtocol { GEODE_FRIEND_MODIFY public: - GEODE_MONOSTATE_CONSTRUCTOR_BEGIN(CCApplication) + GEODE_CUSTOM_CONSTRUCTOR_BEGIN(CCApplication) CCApplication(); virtual ~CCApplication(); diff --git a/loader/include/Geode/cocos/platform/win32/CCApplication.h b/loader/include/Geode/cocos/platform/win32/CCApplication.h index 86756c2e..dd5152da 100644 --- a/loader/include/Geode/cocos/platform/win32/CCApplication.h +++ b/loader/include/Geode/cocos/platform/win32/CCApplication.h @@ -15,7 +15,7 @@ class CC_DLL CCApplication : public CCApplicationProtocol { GEODE_FRIEND_MODIFY public: - GEODE_MONOSTATE_CONSTRUCTOR_BEGIN(CCApplication) + GEODE_CUSTOM_CONSTRUCTOR_BEGIN(CCApplication) CCApplication(); virtual ~CCApplication(); diff --git a/loader/include/Geode/cocos/platform/win32/CCEGLView.h b/loader/include/Geode/cocos/platform/win32/CCEGLView.h index a8a7b838..15a0f65f 100644 --- a/loader/include/Geode/cocos/platform/win32/CCEGLView.h +++ b/loader/include/Geode/cocos/platform/win32/CCEGLView.h @@ -45,6 +45,7 @@ protected: RT_ADD( virtual ~CCEGLView(); ) public: CCEGLView(); + GEODE_CUSTOM_CONSTRUCTOR_COCOS(CCEGLView, CCObject) RT_REMOVE( virtual ~CCEGLView(); ) /* override functions */ diff --git a/loader/include/Geode/cocos/robtop/keyboard_dispatcher/CCKeyboardDelegate.h b/loader/include/Geode/cocos/robtop/keyboard_dispatcher/CCKeyboardDelegate.h index 094bf047..10005f4e 100644 --- a/loader/include/Geode/cocos/robtop/keyboard_dispatcher/CCKeyboardDelegate.h +++ b/loader/include/Geode/cocos/robtop/keyboard_dispatcher/CCKeyboardDelegate.h @@ -207,7 +207,7 @@ RT_ADD( class CC_DLL CCKeyboardHandler : public CCObject { public: - GEODE_MONOSTATE_CONSTRUCTOR_COCOS(CCKeyboardHandler, CCObject) + GEODE_CUSTOM_CONSTRUCTOR_COCOS(CCKeyboardHandler, CCObject) inline CCKeyboardHandler() = default; virtual ~CCKeyboardHandler(); diff --git a/loader/include/Geode/cocos/robtop/mouse_dispatcher/CCMouseDelegate.h b/loader/include/Geode/cocos/robtop/mouse_dispatcher/CCMouseDelegate.h index 0f98200c..4d1f1846 100644 --- a/loader/include/Geode/cocos/robtop/mouse_dispatcher/CCMouseDelegate.h +++ b/loader/include/Geode/cocos/robtop/mouse_dispatcher/CCMouseDelegate.h @@ -22,7 +22,7 @@ RT_ADD( class CC_DLL CCMouseHandler : public CCObject { public: - GEODE_MONOSTATE_CONSTRUCTOR_COCOS(CCMouseHandler, CCObject) + GEODE_CUSTOM_CONSTRUCTOR_COCOS(CCMouseHandler, CCObject) inline CCMouseHandler() = default; virtual ~CCMouseHandler(); diff --git a/loader/include/Geode/cocos/sprite_nodes/CCSprite.h b/loader/include/Geode/cocos/sprite_nodes/CCSprite.h index e633b358..0b4d1702 100644 --- a/loader/include/Geode/cocos/sprite_nodes/CCSprite.h +++ b/loader/include/Geode/cocos/sprite_nodes/CCSprite.h @@ -168,7 +168,7 @@ public: * @js ctor */ CCSprite(void); - GEODE_MONOSTATE_CONSTRUCTOR_COCOS(CCSprite, CCNodeRGBA) + GEODE_CUSTOM_CONSTRUCTOR_COCOS(CCSprite, CCNodeRGBA) /** * Default destructor diff --git a/loader/include/Geode/cocos/sprite_nodes/CCSpriteBatchNode.h b/loader/include/Geode/cocos/sprite_nodes/CCSpriteBatchNode.h index 619eae92..3d2531b3 100644 --- a/loader/include/Geode/cocos/sprite_nodes/CCSpriteBatchNode.h +++ b/loader/include/Geode/cocos/sprite_nodes/CCSpriteBatchNode.h @@ -68,7 +68,7 @@ public: * @js ctor */ CCSpriteBatchNode(); - GEODE_MONOSTATE_CONSTRUCTOR_COCOS(CCSpriteBatchNode, CCNode) + GEODE_CUSTOM_CONSTRUCTOR_COCOS(CCSpriteBatchNode, CCNode) /** * @js NA * @lua NA diff --git a/loader/include/Geode/cocos/text_input_node/CCIMEDelegate.h b/loader/include/Geode/cocos/text_input_node/CCIMEDelegate.h index adff3b12..786d30d4 100644 --- a/loader/include/Geode/cocos/text_input_node/CCIMEDelegate.h +++ b/loader/include/Geode/cocos/text_input_node/CCIMEDelegate.h @@ -50,6 +50,7 @@ class CC_DLL CCIMEDelegate { GEODE_FRIEND_MODIFY public: + GEODE_CUSTOM_CONSTRUCTOR_BEGIN(CCIMEDelegate) virtual ~CCIMEDelegate(); virtual bool attachWithIME(); diff --git a/loader/include/Geode/cocos/textures/CCTexture2D.h b/loader/include/Geode/cocos/textures/CCTexture2D.h index 6a107a48..3ad811ea 100644 --- a/loader/include/Geode/cocos/textures/CCTexture2D.h +++ b/loader/include/Geode/cocos/textures/CCTexture2D.h @@ -117,7 +117,7 @@ public: * @js ctor */ CCTexture2D(); - GEODE_MONOSTATE_CONSTRUCTOR_COCOS(CCTexture2D, CCObject) + GEODE_CUSTOM_CONSTRUCTOR_COCOS(CCTexture2D, CCObject) /** * @js NA * @lua NA diff --git a/loader/include/Geode/cocos/touch_dispatcher/CCTouchHandler.h b/loader/include/Geode/cocos/touch_dispatcher/CCTouchHandler.h index fb818ae4..08b29f9d 100644 --- a/loader/include/Geode/cocos/touch_dispatcher/CCTouchHandler.h +++ b/loader/include/Geode/cocos/touch_dispatcher/CCTouchHandler.h @@ -48,7 +48,7 @@ class CC_DLL CCTouchHandler : public CCObject { GEODE_FRIEND_MODIFY public: - GEODE_MONOSTATE_CONSTRUCTOR_COCOS(CCTouchHandler, CCObject) + GEODE_CUSTOM_CONSTRUCTOR_COCOS(CCTouchHandler, CCObject) inline CCTouchHandler() = default; virtual ~CCTouchHandler(void); @@ -86,7 +86,7 @@ class CC_DLL CCStandardTouchHandler : public CCTouchHandler { GEODE_FRIEND_MODIFY public: - GEODE_MONOSTATE_CONSTRUCTOR_COCOS(CCStandardTouchHandler, CCTouchHandler) + GEODE_CUSTOM_CONSTRUCTOR_COCOS(CCStandardTouchHandler, CCTouchHandler) inline CCStandardTouchHandler() = default; ~CCStandardTouchHandler(void); @@ -110,7 +110,7 @@ class CC_DLL CCTargetedTouchHandler : public CCTouchHandler { GEODE_FRIEND_MODIFY public: - GEODE_MONOSTATE_CONSTRUCTOR_COCOS(CCTargetedTouchHandler, CCTouchHandler) + GEODE_CUSTOM_CONSTRUCTOR_COCOS(CCTargetedTouchHandler, CCTouchHandler) inline CCTargetedTouchHandler() = default; ~CCTargetedTouchHandler(void); diff --git a/loader/include/Geode/modify/Field.hpp b/loader/include/Geode/modify/Field.hpp index c87b59d4..f9374022 100644 --- a/loader/include/Geode/modify/Field.hpp +++ b/loader/include/Geode/modify/Field.hpp @@ -32,8 +32,8 @@ namespace geode::modifier { void* getField(size_t index) { if (m_containedFields.size() <= index) { - m_containedFields.resize(index + 1); - m_destructorFunctions.resize(index + 1); + m_containedFields.push_back(nullptr); + m_destructorFunctions.push_back(nullptr); } return m_containedFields.at(index); } @@ -49,7 +49,9 @@ namespace geode::modifier { } }; + [[deprecated("Will be removed in 1.0.0")]] GEODE_DLL size_t getFieldIndexForClass(size_t hash); + GEODE_DLL size_t getFieldIndexForClass(char const* name); template class FieldIntermediate { @@ -89,8 +91,7 @@ namespace geode::modifier { static_cast(parent)->Parent::~Parent(); } - template > - Parent* operator->() { + operator Parent*() { // get the this pointer of the base // field intermediate is the first member of Modify // meaning we canget the base from ourself @@ -102,7 +103,7 @@ namespace geode::modifier { // the index is global across all mods, so the // function is defined in the loader source - static size_t index = getFieldIndexForClass(typeid(Base).hash_code()); + static size_t index = getFieldIndexForClass(typeid(Base).name()); // the fields are actually offset from their original // offset, this is done to save on allocation and space @@ -119,6 +120,14 @@ namespace geode::modifier { reinterpret_cast(offsetField) - sizeof(Intermediate) ); } + + Parent* self() { + return this->operator Parent*(); + } + + Parent* operator->() { + return this->operator Parent*(); + } }; } diff --git a/loader/include/Geode/modify/Modify.hpp b/loader/include/Geode/modify/Modify.hpp index 925b6757..e5cd1dd1 100644 --- a/loader/include/Geode/modify/Modify.hpp +++ b/loader/include/Geode/modify/Modify.hpp @@ -134,8 +134,8 @@ namespace geode { // abusing the internal stuff // basically we dont want modify to invoke base ctors and dtors // we already have utilities for these, which are ccdestructor - // and the monostate constructor - Modify() : Base(std::monostate(), sizeof(Base)) {} + // and the cutoff constructor + Modify() : Base(CutoffConstructor, sizeof(Base)) {} ~Modify() { cocos2d::CCDestructor::lock(this) = true; diff --git a/loader/include/Geode/utils/Result.hpp b/loader/include/Geode/utils/Result.hpp index 1a9d124f..69cfb5eb 100644 --- a/loader/include/Geode/utils/Result.hpp +++ b/loader/include/Geode/utils/Result.hpp @@ -8,6 +8,7 @@ #include #include #include +#include namespace geode { namespace impl { diff --git a/loader/include/Geode/utils/VersionInfo.hpp b/loader/include/Geode/utils/VersionInfo.hpp index d9df8c88..3fce41e8 100644 --- a/loader/include/Geode/utils/VersionInfo.hpp +++ b/loader/include/Geode/utils/VersionInfo.hpp @@ -56,8 +56,8 @@ namespace geode { constexpr bool operator>(VersionTag const& other) const { if (value == other.value) { if (number && other.number) return number > other.number; - if (number) return true; - if (other.number) return false; + if (number) return false; + if (other.number) return true; return false; } return value > other.value; @@ -65,8 +65,8 @@ namespace geode { constexpr bool operator>=(VersionTag const& other) const { if (value == other.value) { if (number && other.number) return number >= other.number; - if (number) return true; - if (other.number) return false; + if (number) return false; + if (other.number) return true; return true; } return value >= other.value; @@ -77,6 +77,34 @@ namespace geode { std::string toString() const; }; + constexpr bool operator<(std::optional const& a, std::optional const& b) { + if (a && b) return *a < *b; + if (a) return true; + if (b) return false; + return false; + } + + constexpr bool operator<=(std::optional const& a, std::optional const& b) { + if (a && b) return *a <= *b; + if (a) return true; + if (b) return false; + return true; + } + + constexpr bool operator>(std::optional const& a, std::optional const& b) { + if (a && b) return *a > *b; + if (a) return false; + if (b) return true; + return false; + } + + constexpr bool operator>=(std::optional const& a, std::optional const& b) { + if (a && b) return *a >= *b; + if (a) return false; + if (b) return true; + return true; + } + /** * Class representing version information. Uses a limited subset of SemVer; * identifiers are restricted to a few predefined ones, and only one @@ -202,4 +230,4 @@ struct json::Serialize { } return ver.unwrap(); } -}; \ No newline at end of file +}; diff --git a/loader/include/Geode/utils/addresser.hpp b/loader/include/Geode/utils/addresser.hpp index ec09837d..f8f0ffc2 100644 --- a/loader/include/Geode/utils/addresser.hpp +++ b/loader/include/Geode/utils/addresser.hpp @@ -9,36 +9,24 @@ #include #include +#include #include #include #include -#include namespace geode::addresser { - template - inline intptr_t getVirtual(T func); + template + intptr_t getVirtual(Function func); - template - inline intptr_t getNonVirtual(T func); + template + intptr_t getNonVirtual(Function func); - template - inline F thunkAdjust(T func, F self); + template + Class thunkAdjust(Function func, Class self); - template - inline F rthunkAdjust(T func, F self); - - template - Class* friendCreate(typename std::void_t(&Class::create))>*) { - auto ret = Class::create(); - ret->retain(); - return ret; - } - - template - concept HasCreate = requires() { - { friendCreate(nullptr) } -> std::same_as; - }; + template + Class rthunkAdjust(Function func, Class self); class GEODE_DLL Addresser final { template @@ -76,57 +64,20 @@ namespace geode::addresser { return thunk; } - // I gave up - // template - // static Class* generateInstance(Class*) { - // return friendCreate(nullptr); - // } - - // I extra gave up - template - static cocos2d::extension::CCScrollView* generateInstance(cocos2d::extension::CCScrollView*) { - return cocos2d::extension::CCScrollView::create({0.0f, 0.0f}, cocos2d::CCLayer::create()); - } - - template - static cocos2d::CCFileUtils* generateInstance(cocos2d::CCFileUtils*) { - return cocos2d::CCFileUtils::sharedFileUtils(); - } - - template - static Class* generateInstance(Class*) { - if constexpr (std::is_abstract_v) { - // Cant construct abstract classes, so fail early - return nullptr; - } else { - // Create a random memory block with the size of Class - // Assign a pointer to that block and cast it to type Class* - uint8_t dum[sizeof(Class)]{}; - auto ptr = reinterpret_cast(dum); - // Now you have a object of Class that actually isn't an object of Class and is just a - // random memory But C++ doesn't know that of course So now you can copy an object - // that wasn't there in the first place - // ((oh also get the offsets of the virtual tables)) - auto ins = new Class(*ptr); - // this is how the first human was made - return ins; - } - } - template static Class* cachedInstance() { - static auto ret = generateInstance(static_cast(nullptr)); + static auto ret = new Class(ZeroConstructor); return ret; } /** * Specialized functionss */ - template - static intptr_t addressOfVirtual(R (T::*func)(Ps...)) { + template + static intptr_t addressOfVirtual(Return (Class::*func)(Parameters...)) { using geode::cast::reference_cast; - auto ins = cachedInstance(); + auto ins = cachedInstance(); // generateInstance will return nullptr on most abstract classes, // so dont bother getting the address if (ins == nullptr) { @@ -136,97 +87,87 @@ namespace geode::addresser { auto thunk = thunkOf(func); // [[this + thunk] + offset] is the function we want - auto address = *reinterpret_cast(*reinterpret_cast(reinterpret_cast(ins) + thunk) + index); + auto address = *reinterpret_cast( + *reinterpret_cast(reinterpret_cast(ins) + thunk) + index + ); address = followThunkFunction(address); return address; } - template - static intptr_t addressOfVirtual(R (T::*func)(Ps...) const) { - return addressOfVirtual(reinterpret_cast(func)); + template + static intptr_t addressOfVirtual(Return (Class::*func)(Parameters...) const) { + return addressOfVirtual(reinterpret_cast(func)); } - template - static intptr_t addressOfNonVirtual(R (T::*func)(Ps...) const) { - return addressOfNonVirtual(reinterpret_cast(func)); + template + static intptr_t addressOfNonVirtual(Return (Class::*func)(Parameters...) const) { + return addressOfNonVirtual(reinterpret_cast(func)); } static intptr_t followThunkFunction(intptr_t address); - template - static intptr_t addressOfNonVirtual(R (T::*func)(Ps...)) { + template + static intptr_t addressOfNonVirtual(Return (Class::*func)(Parameters...)) { return followThunkFunction(geode::cast::reference_cast(func)); } - template - static intptr_t addressOfNonVirtual(R (*func)(Ps...)) { + template + static intptr_t addressOfNonVirtual(Return (*func)(Parameters...)) { return followThunkFunction(geode::cast::reference_cast(func)); } - template - friend intptr_t getVirtual(T func); + template + friend intptr_t getVirtual(Function func); - template - friend intptr_t getNonVirtual(T func); + template + friend intptr_t getNonVirtual(Function func); - template - friend F thunkAdjust(T func, F self); + template + friend Class thunkAdjust(Function func, Class self); - template - friend F rthunkAdjust(T func, F self); + template + friend Class rthunkAdjust(Function func, Class self); }; -#ifdef GEODE_ADDRESSER_TEST - template - inline intptr_t getVirtual(T func) { + /** + * Gets the real address of a virtual function + */ + template + inline intptr_t getVirtual(Function func) { return Addresser::addressOfVirtual(func); } - template - inline intptr_t getNonVirtual(T func) { + /** + * Gets the real address of a non-virtual function + */ + template + inline intptr_t getNonVirtual(Function func) { return Addresser::addressOfNonVirtual(func); } -#else - template - inline intptr_t getVirtual(T func) { - // log::debug( - // "Get virtual function address from {}", - // utils::intToHex(geode::cast::reference_cast(func)) - // ); - auto addr = Addresser::addressOfVirtual(func); - // log::debug("The address is: {}", utils::intToHex(addr)); - return addr; - } - template - inline intptr_t getNonVirtual(T func) { - // log::debug( - // "Get non-virtual function address from {}", - // utils::intToHex(geode::cast::reference_cast(func)) - // ); - auto addr = Addresser::addressOfNonVirtual(func); - // log::debug("The address is: {}", utils::intToHex(addr)); - return addr; - } - - template - inline F thunkAdjust(T func, F self) { + /** + * Adjusts a class instance to its appropriate base for the given virtual function + */ + template + inline Class thunkAdjust(Function func, Class self) { // do NOT delete the line below. // doing so breaks thunk adjusting on windows. // why? bruh idk auto _ = *geode::cast::template union_cast(&func); - return (F)((intptr_t)self + Addresser::thunkOf(func)); + return (Class)((intptr_t)self + Addresser::thunkOf(func)); } - template - inline F rthunkAdjust(T func, F self) { + /** + * Adjusts a class instance back from its appropriate base for the given virtual function to the original base + */ + template + inline Class rthunkAdjust(Function func, Class self) { // do NOT delete the line below. // doing so breaks thunk adjusting on windows. // why? bruh idk auto _ = *geode::cast::template union_cast(&func); - return (F)((intptr_t)self - Addresser::thunkOf(func)); + return (Class)((intptr_t)self - Addresser::thunkOf(func)); } -#endif } diff --git a/loader/include/link/gdstring.lib b/loader/include/link/gdstring.lib index 11aac0c5..d422bb31 100644 Binary files a/loader/include/link/gdstring.lib and b/loader/include/link/gdstring.lib differ diff --git a/loader/src/cocos2d-ext/Layout.cpp b/loader/src/cocos2d-ext/Layout.cpp index e829754e..c4e64a5f 100644 --- a/loader/src/cocos2d-ext/Layout.cpp +++ b/loader/src/cocos2d-ext/Layout.cpp @@ -9,6 +9,9 @@ USE_GEODE_NAMESPACE(); #pragma warning(disable: 4273) +// if 5k iterations isn't enough to fit the layout, then something is wrong +static size_t RECURSION_DEPTH_LIMIT = 5000; + void CCNode::swapChildIndices(CCNode* first, CCNode* second) { m_pChildren->exchangeObject(first, second); std::swap(first->m_nZOrder, second->m_nZOrder); @@ -277,11 +280,10 @@ AxisLayout::Row* AxisLayout::fitInRow( // also force at least one object to be added to this row, because if // it's too large for this row it's gonna be too large for all rows if ( - m_growCrossAxis && (( + m_growCrossAxis && ( (nextAxisScalableLength + nextAxisUnscalableLength > available.axisLength) && - ix != 0 && - !isOptsSameLine(opts) - )) + ix != 0 && !isOptsSameLine(opts) + ) ) { break; } @@ -323,16 +325,17 @@ AxisLayout::Row* AxisLayout::fitInRow( nodes->removeFirstObject(); } - auto scaleDownFactor = scale - .025f; + // todo: make this calculation more smart to avoid so much unnecessary recursion + auto scaleDownFactor = scale - .0125f; auto squishFactor = available.axisLength / (axisUnsquishedLength + .01f) * squish; // calculate row scale, squish, and prio int tries = 1000; while (axisLength > available.axisLength) { if (this->canTryScalingDown( - res, prio, scale, scale - .025f, minMaxPrios + res, prio, scale, scale - .0125f, minMaxPrios )) { - scale -= .025f; + scale -= .0125f; } else { squish = available.axisLength / (axisUnsquishedLength + .01f) * squish; @@ -366,7 +369,6 @@ AxisLayout::Row* AxisLayout::fitInRow( // the .01f is because floating point arithmetic is imprecise and you // end up in a situation where it confidently tells you that // 241 > 241 == true - // todo: make this calculation more smart to avoid so much unnecessary recursion scaleDownFactor, // how much should the nodes be squished to fit the next item in this // row @@ -389,9 +391,9 @@ bool AxisLayout::canTryScalingDown( // if the scale is less than the lowest min scale allowed, then // trying to scale will have no effect and not help anywmore crossScaleDownFactor < minScaleForPrio || - // if the scale down factor is the same as before, then we've - // entered an infinite loop - crossScaleDownFactor == scale + // if the scale down factor is really close to the same as before, + // then we've entered an infinite loop (float == float is unreliable) + (fabsf(crossScaleDownFactor - scale) < .001f) ) { // is there still some lower priority nodes we could try scaling? if (prio > minMaxPrios.first) { @@ -414,6 +416,7 @@ bool AxisLayout::canTryScalingDown( // otherwise scale as usual else { attemptRescale = true; + scale = crossScaleDownFactor; } return attemptRescale; } @@ -422,7 +425,8 @@ void AxisLayout::tryFitLayout( CCNode* on, CCArray* nodes, std::pair const& minMaxPrios, bool doAutoScale, - float scale, float squish, int prio + float scale, float squish, int prio, + size_t depth ) const { // where do all of these magical calculations come from? // idk i got tired of doing the math but they work so ¯\_(ツ)_/¯ @@ -447,13 +451,13 @@ void AxisLayout::tryFitLayout( rows->addObject(row); if ( row->nextOverflowScaleDownFactor > crossScaleDownFactor && - row->nextOverflowScaleDownFactor <= scale + row->nextOverflowScaleDownFactor < scale ) { crossScaleDownFactor = row->nextOverflowScaleDownFactor; } if ( row->nextOverflowSquishFactor > crossSquishFactor && - row->nextOverflowSquishFactor <= squish + row->nextOverflowSquishFactor < squish ) { crossSquishFactor = row->nextOverflowSquishFactor; } @@ -484,7 +488,8 @@ void AxisLayout::tryFitLayout( if ( !m_allowCrossAxisOverflow && doAutoScale && - totalRowCrossLength > available.crossLength + totalRowCrossLength > available.crossLength && + depth < RECURSION_DEPTH_LIMIT ) { if (this->canTryScalingDown( nodes, prio, scale, crossScaleDownFactor, minMaxPrios @@ -493,7 +498,8 @@ void AxisLayout::tryFitLayout( return this->tryFitLayout( on, nodes, minMaxPrios, doAutoScale, - scale, squish, prio + scale, squish, prio, + depth + 1 ); } } @@ -501,7 +507,8 @@ void AxisLayout::tryFitLayout( // if we're still overflowing, squeeze nodes closer together if ( !m_allowCrossAxisOverflow && - totalRowCrossLength > available.crossLength + totalRowCrossLength > available.crossLength && + depth < RECURSION_DEPTH_LIMIT ) { // if squishing rows would take less squishing that squishing columns, // then squish rows @@ -513,7 +520,8 @@ void AxisLayout::tryFitLayout( return this->tryFitLayout( on, nodes, minMaxPrios, doAutoScale, - scale, crossSquishFactor, prio + scale, crossSquishFactor, prio, + depth + 1 ); } } @@ -719,7 +727,8 @@ void AxisLayout::apply(CCNode* on) { this->tryFitLayout( on, nodes, minMaxPrio, doAutoScale, - this->maxScaleForPrio(nodes, minMaxPrio.second), 1.f, minMaxPrio.second + this->maxScaleForPrio(nodes, minMaxPrio.second), 1.f, minMaxPrio.second, + 0 ); } diff --git a/loader/src/hooks/GeodeNodeMetadata.cpp b/loader/src/hooks/GeodeNodeMetadata.cpp index f520a98b..cf2777ac 100644 --- a/loader/src/hooks/GeodeNodeMetadata.cpp +++ b/loader/src/hooks/GeodeNodeMetadata.cpp @@ -79,9 +79,13 @@ struct ProxyCCNode : Modify { } }; -static inline std::unordered_map s_nextIndex; +static inline std::unordered_map s_nextIndex; +size_t modifier::getFieldIndexForClass(char const* name) { + return s_nextIndex[name]++; +} + size_t modifier::getFieldIndexForClass(size_t hash) { - return s_nextIndex[hash]++; + return s_nextIndex[std::to_string(hash)]++; } // not const because might modify contents diff --git a/loader/src/ids/EditorPauseLayer.cpp b/loader/src/ids/EditorPauseLayer.cpp index 6b43b69b..00b96c68 100644 --- a/loader/src/ids/EditorPauseLayer.cpp +++ b/loader/src/ids/EditorPauseLayer.cpp @@ -237,6 +237,7 @@ $register_ids(EditorPauseLayer) { ->setAxisAlignment(AxisAlignment::End) ->setAxisReverse(true) ->setCrossAxisOverflow(false) + ->setCrossAxisAlignment(AxisAlignment::Start) ->setCrossAxisLineAlignment(AxisAlignment::Start), this->getChildByID("object-count-label"), this->getChildByID("length-label"), @@ -250,8 +251,8 @@ $register_ids(EditorPauseLayer) { ->setBreakLine(true) ); } - menu->setContentSize({ 165.f, 100.f }); - menu->setPosition(75.f, winSize.height - 55.f); + menu->setContentSize({ 180.f, 100.f }); + menu->setPosition(100.f, winSize.height - 55.f); menu->updateLayout(); } } diff --git a/loader/src/loader/ModImpl.cpp b/loader/src/loader/ModImpl.cpp index d5e8f359..6c0146d4 100644 --- a/loader/src/loader/ModImpl.cpp +++ b/loader/src/loader/ModImpl.cpp @@ -356,7 +356,9 @@ Result<> Mod::Impl::enable() { } for (auto const& hook : m_hooks) { - GEODE_UNWRAP(this->enableHook(hook)); + if (hook->getAutoEnable()) { + GEODE_UNWRAP(this->enableHook(hook)); + } } for (auto const& patch : m_patches) {