Re: [PATCH v10 0/1] MR7322: programs/cmd: Implement ability to abort lengthy directory operations via Ctrl-C.
16 Feb
2025
16 Feb
'25
4:38 p.m.
eric pouech (@epo) commented about programs/cmd/directory.c:
* FIXME: Assumes 24-line display for the /P qualifier. */
-static DIRECTORY_STACK *WCMD_list_directory (DIRECTORY_STACK *inputparms, int level) { +static DIRECTORY_STACK *WCMD_list_directory (DIRECTORY_STACK *inputparms, int level, RETURN_CODE *return_code) { nitpick: IMO it's more readable to return a RETURN_CODE, and pass a DIRECTORY_STACK\*\* to update the pointer (you likely did it this way to minimize change, but let's take the opportunity of a change to sanitize a bit the current spaghetti code)
-- https://gitlab.winehq.org/wine/wine/-/merge_requests/7322#note_94822
307
Age (days ago)
307
Last active (days ago)
0 comments
1 participants
participants (1)
-
eric pouech (@epo)