pafuhana1213 / KawaiiPhysics

KawaiiPhysics : Simple fake Physics for UnrealEngine4 & 5
MIT License
1.92k stars 287 forks source link

ver5.4 #116

Closed hacuwork closed 6 months ago

hacuwork commented 6 months ago

ver5.4にどうしたら対応させられますか?

SeethingNikuJu commented 6 months ago

私の UE5.3 プロジェクト(KawaiiPhysics Version 1.14.0)を UE5.4 にバージョンアップするためにソースビルドを行うと、C4702 エラーが発生しました。

...
[7/14] Compile [x64] Module.KawaiiPhysics.cpp
D:\UnrealEngine\MyProject\Plugins\KawaiiPhysics\Source\KawaiiPhysics\Private\AnimNode_KawaiiPhysics.cpp(160) : error C4702: unreachable code
   while compiling FAnimNode_KawaiiPhysics::HasPreUpdate
...

以下の修正でビルドは通りましたが、適切な対応なのかはわかりません。

$ git diff AnimNode_KawaiiPhysics.cpp
diff --git a/Plugins/KawaiiPhysics/Source/KawaiiPhysics/Private/AnimNode_KawaiiPhysics.cpp b/Plugins/KawaiiPhysics/Source/KawaiiPhysics/Private/AnimNode_KawaiiPhysics.cpp
index e76b635..64e58e2 100644
--- a/Plugins/KawaiiPhysics/Source/KawaiiPhysics/Private/AnimNode_KawaiiPhysics.cpp
+++ b/Plugins/KawaiiPhysics/Source/KawaiiPhysics/Private/AnimNode_KawaiiPhysics.cpp
@@ -155,9 +155,9 @@ bool FAnimNode_KawaiiPhysics::HasPreUpdate() const
 {
 #if WITH_EDITOR
        return true;
-#endif
-
+#else
        return false;
+#endif
 }

 void FAnimNode_KawaiiPhysics::PreUpdate(const UAnimInstance* InAnimInstance)

今のところ、KawaiiPhysics は UE5.3 と同じく正しく動いているように見えます。

pafuhana1213 commented 6 months ago

@SeethingNikuJu 修正コードをありがとうございます! ご共有いただいた対応で問題ないかと思いますので、取り込んだものを以下でコミットしました! https://github.com/pafuhana1213/KawaiiPhysics/commit/89d1c86b00ed4b3b18fe62fd673c9718d9afc9b0

@hacuwork UE5.4対応版をこちらにコミットしました! https://github.com/pafuhana1213/KawaiiPhysics/discussions/117