updated: [3d0f9e4] Reverted some more GString removals. (See f235b1976ee6dd7aa2be7e75c870784c424e3de3)
Patrick Winnertz
winnie at debian.org
Tue Feb 10 11:49:53 UTC 2009
The following commit has been merged in the master branch:
commit 3d0f9e419a1b8a28196ee15b93adf56eafdd1a1e
Author: Patrick Winnertz <winnie at debian.org>
Date: Thu Feb 5 23:22:08 2009 +0100
Reverted some more GString removals. (See f235b1976ee6dd7aa2be7e75c870784c424e3de3)
Signed-off-by: Patrick Winnertz <winnie at debian.org>
diff --git a/edit/edit.c b/edit/edit.c
index e853a96..4504df7 100644
--- a/edit/edit.c
+++ b/edit/edit.c
@@ -153,9 +153,10 @@ edit_load_file_fast (WEdit *edit, const char *filename)
buf2 = edit->curs2 >> S_EDIT_BUF_SIZE;
if ((file = mc_open (filename, O_RDONLY | O_BINARY)) == -1) {
- char errmsg[8192];
- snprintf(errmsg, sizeof(errmsg), _(" Cannot open %s for reading "), filename);
- edit_error_dialog (_("Error"), get_sys_error (errmsg));
+ GString *errmsg = g_string_new(NULL);
+ g_string_sprintf(errmsg, _(" Cannot open %s for reading "), filename);
+ edit_error_dialog (_("Error"), get_sys_error (errmsg->str));
+ g_string_free (errmsg, TRUE);
return 1;
}
@@ -315,8 +316,7 @@ static int
check_file_access (WEdit *edit, const char *filename, struct stat *st)
{
int file;
- char errmsg[8192];
- errmsg[0] = 0;
+ GString *errmsg = (GString *) 0;
/* Try opening an existing file */
file = mc_open (filename, O_NONBLOCK | O_RDONLY | O_BINARY, 0666);
@@ -331,7 +331,8 @@ check_file_access (WEdit *edit, const char *filename, struct stat *st)
O_NONBLOCK | O_RDONLY | O_BINARY | O_CREAT | O_EXCL,
0666);
if (file < 0) {
- snprintf (errmsg, sizeof(errmsg), _(" Cannot open %s for reading "), filename);
+ g_string_sprintf (errmsg = g_string_new (NULL),
+ _(" Cannot open %s for reading "), filename);
goto cleanup;
} else {
/* New file, delete it if it's not modified or saved */
@@ -341,13 +342,15 @@ check_file_access (WEdit *edit, const char *filename, struct stat *st)
/* Check what we have opened */
if (mc_fstat (file, st) < 0) {
- snprintf (errmsg, sizeof(errmsg), _(" Cannot get size/permissions for %s "), filename);
+ g_string_sprintf (errmsg = g_string_new (NULL),
+ _(" Cannot get size/permissions for %s "), filename);
goto cleanup;
}
/* We want to open regular files only */
if (!S_ISREG (st->st_mode)) {
- snprintf (errmsg, sizeof(errmsg), _(" %s is not a regular file "), filename);
+ g_string_sprintf (errmsg = g_string_new (NULL),
+ _(" %s is not a regular file "), filename);
goto cleanup;
}
@@ -360,14 +363,16 @@ check_file_access (WEdit *edit, const char *filename, struct stat *st)
}
if (st->st_size >= SIZE_LIMIT) {
- snprintf (errmsg, sizeof(errmsg), _(" File %s is too large "), filename);
+ g_string_sprintf (errmsg = g_string_new (NULL),
+ _(" File %s is too large "), filename);
goto cleanup;
}
cleanup:
(void) mc_close (file);
- if (errmsg[0]) {
- edit_error_dialog (_("Error"), errmsg);
+ if (errmsg) {
+ edit_error_dialog (_("Error"), errmsg->str);
+ g_string_free (errmsg, TRUE);
return 1;
}
return 0;
diff --git a/edit/editcmd.c b/edit/editcmd.c
index 8de68ea..9d730e0 100644
--- a/edit/editcmd.c
+++ b/edit/editcmd.c
@@ -1214,16 +1214,14 @@ edit_replace_prompt (WEdit * edit, char *replace_text, int xpos, int ypos)
0, 0, 0, 0, 0},
NULL_QuickWidget};
- const char* label_nls = _(" Replace with: ");
- char label_text[8192];
- if (*replace_text)
- {
- size_t label_len = strlen(label_nls);
- snprintf(label_text, sizeof(label_text), "%s%s", label_nls, replace_text);
- convert_to_display((&label_text)+label_len);
+ GString *label_text = g_string_new (_(" Replace with: "));
+ if (*replace_text) {
+ size_t label_len;
+ label_len = label_text->len;
+ g_string_append (label_text, replace_text);
+ convert_to_display (label_text->str + label_len);
}
-
- quick_widgets[5].text = label_text;
+ quick_widgets[5].text = label_text->str;
{
int retval;
@@ -1242,6 +1240,7 @@ edit_replace_prompt (WEdit * edit, char *replace_text, int xpos, int ypos)
Quick_input.ypos = ypos;
retval = quick_dialog (&Quick_input);
+ g_string_free (label_text, TRUE);
return retval;
}
}
diff --git a/edit/syntax.c b/edit/syntax.c
index 5de3aaf..479f3c5 100644
--- a/edit/syntax.c
+++ b/edit/syntax.c
@@ -505,14 +505,10 @@ void edit_get_syntax_color (WEdit * edit, long byte_index, int *color)
*/
static int read_one_line (char **line, FILE * f)
{
- char buffer[8192];
- int index = 0, c, r = 0;
- buffer[0] = 0;
+ GString *p = g_string_new ("");
+ int c, r = 0;
for (;;) {
- if (index >= (sizeof(buffer)-1))
- break;
-
c = fgetc (f);
if (c == EOF) {
if (ferror (f)) {
@@ -534,11 +530,13 @@ static int read_one_line (char **line, FILE * f)
if (c == '\n')
break;
- buffer[index] = c;
- index++;
+ g_string_append_c (p, c);
}
if (r != 0) {
- *line = mhl_str_dup(buffer);
+ *line = p->str;
+ g_string_free (p, FALSE);
+ } else {
+ g_string_free (p, TRUE);
}
return r;
}
--
Midnight Commander Development
More information about the mc-devel
mailing list