New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
jak1: clean up all dummy
methods
#2457
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for doing the cleanup! I just found one missing thing, but I'll grab it in a separate PR.
|
||
|
||
; (defmethod find-edge-grabs! target ((obj target) (arg0 collide-cache)) | ||
; (format *stdcon* "no edge grabs for you!~%") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
wow, I guess this has been here for a while...
) | ||
(else | ||
(new 'static 'rgba :r #xff :g #x80 :a #x40) | ||
) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cool to see that this works now
@@ -451,13 +451,13 @@ | |||
(+! (-> obj sim-time-remaining) | |||
(* 0.0033333334 (the float (- (-> *display* base-frame-counter) (-> *display* old-base-frame-counter)))) | |||
) | |||
(let ((f30-0 (* DISPLAY_FPS_RATIO 0.016666668)) ;; changed for high fps | |||
(let ((f30-0 0.016666668) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing patch
) | ||
(+! (-> obj to-vu0-waits) arg0) | ||
(+! (-> obj to-spr-waits) arg1) | ||
(+! (-> obj from-spr-waits) arg2) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
wow, this must be ancient if +!
wasn't recognized!
Cleans up every
dummy-*
andTODO-RENAME-*
method up with either proper names or by renaming them to[type-name]-method-[method-id]
similar to Jak 2'sall-types
.Also fixes the bad format string in
collide-cache
and adds the event handler hack to Jak 1.The game boots and runs fine, but I might have missed a PAL patch or other manual patches here and there, please double-check if possible.