commit 2b9c0c57241eb55c5cb8a7ba3e0e376cb3af0d6a
parent abfde53945e6e6107994c7c9ab87fcff16d98a25
Author: Benno Schulenberg <bensberg@telfort.nl>
Date: Sat, 4 Jul 2020 15:12:04 +0200
tweaks: rename a variable, and reduce the scope of two others
Diffstat:
1 file changed, 8 insertions(+), 7 deletions(-)
diff --git a/src/browser.c b/src/browser.c
@@ -43,20 +43,18 @@ static size_t selected = 0;
* start browsing from. */
char *do_browser(char *path)
{
- char *retval = NULL;
- int kbinput;
char *present_name = NULL;
/* The name of the currently selected file, or of the directory we
* were in before backing up to "..". */
size_t old_selected;
/* The number of the selected file before the current selected file. */
- functionptrtype func;
- /* The function of the key the user typed in. */
DIR *dir;
/* The directory whose contents we are showing. */
+ char *chosen = NULL;
+ /* The name of the file that the user picked, or NULL if none. */
read_directory_contents:
- /* We come here when we refresh or select a new directory. */
+ /* We come here when the user refreshes or selects a new directory. */
path = free_and_assign(path, get_full_path(path));
@@ -101,6 +99,9 @@ char *do_browser(char *path)
titlebar(path);
while (TRUE) {
+ functionptrtype func;
+ int kbinput;
+
lastmessage = VACUUM;
bottombars(MBROWSER);
@@ -297,7 +298,7 @@ char *do_browser(char *path)
/* If it isn't a directory, a file was selected -- we're done. */
if (!S_ISDIR(st.st_mode)) {
- retval = copy_of(filelist[selected]);
+ chosen = copy_of(filelist[selected]);
break;
}
@@ -342,7 +343,7 @@ char *do_browser(char *path)
filelist = NULL;
filelist_len = 0;
- return retval;
+ return chosen;
}
/* The file browser front end. We check to see if inpath has a