https://gitlab.synchro.net/main/sbbs/-/commit/97dd3955d69d482a48a6010f
Modified Files:
src/sftp/sftp.h sftp_client.c sftp_server.c src/syncterm/sftp_browser.c Log Message:
sftp: drop speculative public API surface with no callers
Eight client ops (sftpc_lstat/fstat/fsetstat/remove/rmdir/mkdir/rename /reclaim) and the sftpc_debug_last_reply_type accessor were declared
in sftp.h and defined in sftp_client.c with no callers anywhere in
the tree (verified: src/sftp, src/syncterm, src/sbbs3). They came in
with f193bd5695 as speculative surface; nothing uses them today, and
the debug accessor only existed to prepend a reply-type line to a
single error string in sftp_browser.c.
sftps_reclaim is internal-only (one self-caller in sftp_server.c);
make it static rather than exposing it as public API.
Also drop the dead state->last_reply_type field, its two writes in sftp_client.c, and the do_one_path helper that was only used by the
removed sftpc_remove/rmdir.
Co-Authored-By: Claude Opus 4.7 (1M context) <
[email protected]>
--- SBBSecho 3.37-Linux
* Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)