Skip to content

field: More field_event functions - #147

Merged
Xeeynamo merged 6 commits into
Xeeynamo:mainfrom
Zaarbs:field-ops
Sep 8, 2026
Merged

field: More field_event functions#147
Xeeynamo merged 6 commits into
Xeeynamo:mainfrom
Zaarbs:field-ops

Conversation

@Zaarbs

@Zaarbs Zaarbs commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

The last 3 functions of field_event.c are for drawing the position cursor above Cloud's head, not the exit arrows. I've updated the relevant names to reflect that. Also added macro for getting entity name and animation state enum.

Comment thread include/psxsdk/libgpu.h
@@ -134,8 +134,10 @@ extern int (*GPU_printf)(); /* printf() object */
((tge) ? setcode(p, getcode(p) | 0x01) : setcode(p, getcode(p) & ~0x01))

#define getTPage(tp, abr, x, y) \

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I replaced the getTPage definition from Psy-Q 4.x with the definition from 3.3-3.6 as ResetPositionCursorPrimitives requires it to match. Didn't find any conflicting usages of it.

Comment thread include/psxsdk/libgpu.h
extern DRAWENV* SetDefDrawEnv(DRAWENV* env, int x, int y, int w, int h);
extern TIM_IMAGE* ReadTIM(TIM_IMAGE* timimg);

extern int GetGraphType(void);

@Zaarbs Zaarbs Sep 7, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

https://psx.arthus.net/sdk/Psy-Q/DOCS/BBS/webmsg.pdf - Page 31 says to add the declaration like this since it wasn't included in LIBGPU.H by default.

@Xeeynamo Xeeynamo left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Spotless 👌

@Xeeynamo
Xeeynamo merged commit bce2d9c into Xeeynamo:main Sep 8, 2026
1 check passed
@Zaarbs
Zaarbs deleted the field-ops branch September 9, 2026 14:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants