Skip to content

feat(libSceNetCtlApDialog): run the common-dialog state machine - #3941

Closed
semirkabir wants to merge 1 commit into
boykopovar:mainfrom
semirkabir:feat/netctl-ap-dialog-state-machine
Closed

semirkabir wants to merge 1 commit into
boykopovar:mainfrom
semirkabir:feat/netctl-ap-dialog-state-machine

Conversation

@semirkabir

Copy link
Copy Markdown
Contributor

What

Runs the access-point dialog through the common-dialog state machine (none, initialized, running, finished), following libSceErrorDialog and libSceLoginDialog. The dialog shows nothing and reports the user as having cancelled. Adds the seventh export sceNetCtlApDialogGetStatus (NID eBWS-HorSWY), a guest test, and a TechnicalDebt entry for the assumed signatures and error codes.

Tested

Linux x86-64, GCC. Built the patched and unpatched libSceNetCtlApDialog.prx (7 exports each) and ran guest_netctl_ap_dialog (ctest passes). Not tested on hardware.

Checklist

  • Based on current main; no other open PR implements the same functions
  • One topic per PR; follow-ups go in a new PR
  • No comments in code except technical debt
  • Unimplemented paths throw (NotImplemented_nid_no_patch); silent stubs are listed in TechnicalDebt
  • No notes, investigation .md files or images added to the repository (attach them to this PR)
  • New third-party code is a submodule built from source
  • Generic behaviour, not specific to one title
  • Depends on: none
  • AI-assisted: yes

The access-point dialog shows nothing and reports the user as having
cancelled, following the libSceErrorDialog and libSceLoginDialog pattern.
Adds the seventh export sceNetCtlApDialogGetStatus (NID eBWS-HorSWY),
whose signature is assumed from the other common dialogs, a guest test,
and a TechnicalDebt entry covering the assumed signatures and error codes.
@github-actions

Copy link
Copy Markdown

Progress report (a11c8f7 → d2d997a)

📈 System libraries: 85.99% (+0.18%, +7 implemented functions)

✅ 7 implemented
Library Name
libSceNetCtlApDialog sceNetCtlApDialogClose
libSceNetCtlApDialog sceNetCtlApDialogGetResult
libSceNetCtlApDialog sceNetCtlApDialogGetStatus
libSceNetCtlApDialog sceNetCtlApDialogInitialize
libSceNetCtlApDialog sceNetCtlApDialogOpen
libSceNetCtlApDialog sceNetCtlApDialogTerminate
libSceNetCtlApDialog sceNetCtlApDialogUpdateStatus
⚪ 1 behaviourally empty (constant return or throw)
Library Name
libSceNetCtlApDialog sceNetCtlApDialogGetResult

@github-actions

github-actions Bot commented Oct 11, 2026 •

Copy link
Copy Markdown

Other open pull requests touch the same code. Merging one will break the other, or both implement the same thing. If this one builds on them, list them in Depends on.

Pull request Conflicting files (lines on main) Same exports Same new files
#4098 core/libs/prx/libSceNetCtlApDialog/Export.cpp L0, L5-6, L8-10, L13-15, L18-20, L23-25, L28-30, L33-35
core/libs/tests/GuestNetCtlApDialog.cpp
sceNetCtlApDialogClose
sceNetCtlApDialogTerminate
core/libs/prx/libSceNetCtlApDialog/CMakeLists.txt
core/libs/tests/GuestNetCtlApDialog.cpp

@oneandonlydean

Copy link
Copy Markdown
Collaborator

Closing: no title is shown reaching libSceNetCtlApDialog, and the no-argument signatures and the error codes are assumed. Reopen with the title ID and the log line where one stops on these exports.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants