debian/notmuch
1
0
Fork 0
mirror of https://git.notmuchmail.org/git/notmuch synced 2024-11-22 19:08:09 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions

notmuch: Use GNU libc getline() instead of glib GIOChannel

Browse source 
Less reliance on glib is always nice for our memory-leak testing
efforts.
This commit is contained in:
Carl Worth 2009-10-20 12:47:23 -07:00
parent 968feafbad
commit 00b65cad98
1 changed files with 11 additions and 38 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

49
notmuch.c
View file

@ -20,8 +20,13 @@
#include "notmuch.h" #include "notmuch.h"
#ifndef _GNU_SOURCE
#define _GNU_SOURCE /* for getline */
#endif
#include <stdio.h> #include <stdio.h>
#include <stdlib.h> #include <stdlib.h>
#include <stddef.h>
#include <string.h> #include <string.h>
#include <sys/types.h> #include <sys/types.h>
#include <sys/stat.h> #include <sys/stat.h>
@ -30,7 +35,7 @@
#include <dirent.h> #include <dirent.h>
#include <errno.h> #include <errno.h>
#include <glib.h> /* GIOChannel */ #include <glib.h> /* g_strdup_printf */
#define ARRAY_SIZE(arr) (sizeof (arr) / sizeof (arr[0])) #define ARRAY_SIZE(arr) (sizeof (arr) / sizeof (arr[0]))
@ -42,42 +47,6 @@ typedef struct command {
const char *usage; const char *usage;
} command_t; } command_t;
/* Read a line from stdin, without any line-terminator character. The
* return value is a newly allocated string. The caller should free()
* the string when finished with it.
*
* This function returns NULL if EOF is encountered before any
* characters are input (otherwise it returns those characters).
*/
char *
read_line (void)
{
char *result = NULL;
GError *error = NULL;
GIOStatus g_io_status;
gsize length;
GIOChannel *channel = g_io_channel_unix_new (fileno (stdin));
g_io_status = g_io_channel_read_line (channel, &result,
&length, NULL, &error);
if (g_io_status == EOF)
goto DONE;
if (g_io_status != G_IO_STATUS_NORMAL) {
fprintf(stderr, "Read error: %s\n", error->message);
exit (1);
}
if (length && result[length - 1] == '\n')
result[length - 1] = '\0';
DONE:
g_io_channel_unref (channel);
return result;
}
typedef struct { typedef struct {
int total_messages; int total_messages;
int count; int count;
@ -286,6 +255,7 @@ setup_command (int argc, char *argv[])
{ {
notmuch_database_t *notmuch; notmuch_database_t *notmuch;
char *mail_directory, *default_path; char *mail_directory, *default_path;
size_t line_size;
int count; int count;
add_files_state_t add_files_state; add_files_state_t add_files_state;
double elapsed; double elapsed;
@ -315,7 +285,8 @@ setup_command (int argc, char *argv[])
printf ("Top-level mail directory [%s]: ", default_path); printf ("Top-level mail directory [%s]: ", default_path);
fflush (stdout); fflush (stdout);
mail_directory = read_line (); mail_directory = NULL;
getline (&mail_directory, &line_size, stdin);
printf ("\n"); printf ("\n");
if (mail_directory == NULL || strlen (mail_directory) == 0) { if (mail_directory == NULL || strlen (mail_directory) == 0) {
@ -323,6 +294,8 @@ setup_command (int argc, char *argv[])
free (mail_directory); free (mail_directory);
mail_directory = default_path; mail_directory = default_path;
} else { } else {
if (mail_directory[strlen(mail_directory)-1] == '\n')
mail_directory[strlen(mail_directory)-1] = '\0';
/* XXX: Instead of telling the user to use an environment /* XXX: Instead of telling the user to use an environment
* variable here, we should really be writing out a configuration * variable here, we should really be writing out a configuration
* file and loading that on the next run. */ * file and loading that on the next run. */