From 24968886824da7d4af308a113d550f4dc97bba48 Mon Sep 17 00:00:00 2001
From: Frederic Culot <calcurse@culot.org>
Date: Wed, 4 Apr 2007 19:38:18 +0000
Subject: MAX_LENGTH replaced by stdio.h's BUFSIZ

---
 src/custom.c | 11 ++++++-----
 src/day.c    | 11 ++++++-----
 src/day.h    |  8 +++++---
 src/help.c   | 11 ++++++-----
 src/io.c     | 36 ++++++++++++++++++------------------
 src/recur.c  | 14 +++++++++-----
 src/todo.c   |  8 ++++----
 src/vars.h   | 20 +++++++++-----------
 8 files changed, 63 insertions(+), 56 deletions(-)

(limited to 'src')

diff --git a/src/custom.c b/src/custom.c
index 73cf362..ce4bbc0 100755
--- a/src/custom.c
+++ b/src/custom.c
@@ -1,4 +1,4 @@
-/*	$calcurse: custom.c,v 1.6 2007/03/24 23:18:52 culot Exp $	*/
+/*	$calcurse: custom.c,v 1.7 2007/04/04 19:38:18 culot Exp $	*/
 
 /*
  * Calcurse - text-based organizer
@@ -27,6 +27,7 @@
 #include <ncurses.h>
 #include <string.h>
 #include <stdlib.h>
+#include <stdio.h>
 
 #include "i18n.h"
 #include "io.h"
@@ -273,7 +274,7 @@ custom_color_config(int notify_bar)
 		NBPOS
 	};
 	WINDOW *conf_win;
-	char label[MAX_LENGTH];
+	char label[BUFSIZ];
 	char *choose_color_1 = 
 	    _("Use 'X' or SPACE to select a color, "
 	    "'H/L' 'J/K' or arrow keys to move");
@@ -312,7 +313,7 @@ custom_color_config(int notify_bar)
 	clear();
 	win_row = (notify_bar) ? row - 3 : row - 2;
 	conf_win = newwin(win_row, col, 0, 0);
-	snprintf(label, MAX_LENGTH, _("CalCurse %s | color theme"), VERSION);
+	snprintf(label, BUFSIZ, _("CalCurse %s | color theme"), VERSION);
 	win_show(conf_win, label);
 	status_mesg(choose_color_1, choose_color_2);
 
@@ -451,7 +452,7 @@ custom_load_color(char *color, int background)
 #define AWAITED_COLORS	2
 
 	int i, len, color_num;
-	char c[AWAITED_COLORS][MAX_LENGTH];
+	char c[AWAITED_COLORS][BUFSIZ];
 	int colr[AWAITED_COLORS];
 	const char *wrong_color_number =
 	    _("FATAL ERROR in custom_load_color: wrong color number.\n");
@@ -588,6 +589,6 @@ custom_color_theme_name(char *theme_name)
 		}
 	}
 
-	snprintf(theme_name, MAX_LENGTH, "%s on %s", color_name[0],
+	snprintf(theme_name, BUFSIZ, "%s on %s", color_name[0],
 	    color_name[1]);
 }
diff --git a/src/day.c b/src/day.c
index 826b332..e4c1ebd 100755
--- a/src/day.c
+++ b/src/day.c
@@ -1,4 +1,4 @@
-/*	$calcurse: day.c,v 1.20 2007/03/10 15:55:25 culot Exp $	*/
+/*	$calcurse: day.c,v 1.21 2007/04/04 19:38:18 culot Exp $	*/
 
 /*
  * Calcurse - text-based organizer
@@ -404,6 +404,7 @@ day_edit_item(int year, int month, int day, int item_num)
 #define END	'2'
 #define DESC	'3'
 #define REPT	'4'
+
 	struct day_item_s *p;
 	struct recur_event_s *re;
 	struct rpt_s *rpt;
@@ -520,7 +521,7 @@ day_edit_item(int year, int month, int day, int item_num)
 		while ( (ch != 'D') && (ch != 'W') && (ch != 'M') 
 		    && (ch != 'Y') ) {
 			status_mesg(mesg_type_1, mesg_type_2);
-			typestr = (char *) malloc(sizeof(char)); 
+			typestr = (char *)malloc(sizeof(char) * 2);
 			*typestr = recur_def2char(rpt->type);
 			cancel = updatestring(swin, &typestr, 0, 1);
 			ch = toupper(*typestr);
@@ -530,8 +531,8 @@ day_edit_item(int year, int month, int day, int item_num)
 		}
 		while (newfreq == 0) {
 			status_mesg(mesg_freq_1, "");
-			freqstr = (char *) malloc(MAX_LENGTH); 
-			snprintf(freqstr, MAX_LENGTH, "%d", rpt->freq);
+			freqstr = (char *) malloc(BUFSIZ); 
+			snprintf(freqstr, BUFSIZ, "%d", rpt->freq);
 			cancel = updatestring(swin, &freqstr, 0, 1);
 			newfreq = atoi(freqstr);
 			free(freqstr);
@@ -553,7 +554,7 @@ day_edit_item(int year, int month, int day, int item_num)
 				return;
 			}
 			if (strlen(timestr) == 1 && 
-			    strncmp(timestr, "0", 1) == 0 )  {
+			    strncmp(timestr, "0", 1) == 0 ) {
 				rpt->until = 0;
 				date_entered = 1;
 			} else { 
diff --git a/src/day.h b/src/day.h
index 2363bc9..c59e813 100755
--- a/src/day.h
+++ b/src/day.h
@@ -1,4 +1,4 @@
-/*	$calcurse: day.h,v 1.10 2007/03/10 15:55:25 culot Exp $	*/
+/*	$calcurse: day.h,v 1.11 2007/04/04 19:41:27 culot Exp $	*/
 
 /*
  * Calcurse - text-based organizer
@@ -27,6 +27,8 @@
 #ifndef CALCURSE_DAY_H
 #define CALCURSE_DAY_H
 
+#include <stdio.h>
+
 #include "vars.h"
 #include "apoint.h"
 
@@ -49,8 +51,8 @@ struct day_item_s {
 };
 
 struct day_saved_item_s {
-	char start[MAX_LENGTH]; 
-	char end[MAX_LENGTH];
+	char start[BUFSIZ]; 
+	char end[BUFSIZ];
 	char state;
 	char type ;
 	char *mesg;
diff --git a/src/help.c b/src/help.c
index 9e06785..e7bc8d5 100755
--- a/src/help.c
+++ b/src/help.c
@@ -1,4 +1,4 @@
-/*	$calcurse: help.c,v 1.15 2007/03/24 23:19:48 culot Exp $	*/
+/*	$calcurse: help.c,v 1.16 2007/04/04 19:38:18 culot Exp $	*/
 
 /*
  * Calcurse - text-based organizer
@@ -26,6 +26,7 @@
 
 #include <ncurses.h>
 #include <stdlib.h>
+#include <stdio.h>
 #include <string.h>
 #include <stdbool.h>
 #include <ctype.h>
@@ -51,7 +52,7 @@ write_help_pad(WINDOW *win, char *title, char *text, int pad_width)
 	
 	nl_text = get_help_lines(text);
 	nl_title = get_help_lines(title);
-	erase_window_part(win, 0, 0, MAX_LENGTH, pad_width);
+	erase_window_part(win, 0, 0, BUFSIZ, pad_width);
 	custom_apply_attr(win, ATTR_HIGHEST);
 	mvwprintw(win, 0, 0, "%s", title);
 	custom_remove_attr(win, ATTR_HIGHEST);
@@ -77,7 +78,7 @@ help_screen(int which_pan)
 {
 	WINDOW *help_win = NULL;
 	WINDOW *help_pad = NULL;
-	char label[MAX_LENGTH];
+	char label[BUFSIZ];
 	int ch = '?';
 	int help_row, text_lines;
 	int help_col = col;
@@ -364,9 +365,9 @@ help_screen(int which_pan)
 	help_row = (notify_bar()) ? row - 3 : row - 2; 
 	text_lines = help_row - (pad_offset + 1);
 	help_win = newwin(help_row, help_col, 0, 0);
-	help_pad = newpad(MAX_LENGTH, pad_width);
+	help_pad = newpad(BUFSIZ, pad_width);
 	box(help_win, 0, 0);
-	snprintf(label, MAX_LENGTH, _("CalCurse %s | help"), VERSION);
+	snprintf(label, BUFSIZ, _("CalCurse %s | help"), VERSION);
 	win_show(help_win, label);
 
 	/* Display the help screen related to user input. */
diff --git a/src/io.c b/src/io.c
index 17ae4b5..3700ad4 100755
--- a/src/io.c
+++ b/src/io.c
@@ -1,4 +1,4 @@
-/*	$calcurse: io.c,v 1.13 2007/03/24 23:20:49 culot Exp $	*/
+/*	$calcurse: io.c,v 1.14 2007/04/04 19:38:18 culot Exp $	*/
 
 /*
  * Calcurse - text-based organizer
@@ -143,11 +143,11 @@ io_get_export_stream(void)
 	    _("Press [ENTER] to continue.");
 
 	stream = NULL;
-	stream_name = (char *)malloc(MAX_LENGTH);
+	stream_name = (char *)malloc(BUFSIZ);
 	home = getenv("HOME");
 	if (home == NULL)
 		home = ".";
-	snprintf(stream_name, MAX_LENGTH, "%s/calcurse.ics", home);
+	snprintf(stream_name, BUFSIZ, "%s/calcurse.ics", home);
 	
 	while (stream == NULL) {
 		status_mesg(question, "");
@@ -198,7 +198,7 @@ io_export_recur_events(FILE *stream)
 {
 	struct recur_event_s *i;
 	struct days_s *day;
-	char ical_date[MAX_LENGTH];
+	char ical_date[BUFSIZ];
 
 	for (i = recur_elist; i != 0; i = i->next) { 
 		date_sec2ical_date(i->day, ical_date);
@@ -233,7 +233,7 @@ void
 io_export_events(FILE *stream)
 {
 	struct event_s *i;
-	char ical_date[MAX_LENGTH];
+	char ical_date[BUFSIZ];
 	
 	for (i = eventlist; i != 0; i = i->next) { 
 		date_sec2ical_date(i->day, ical_date);
@@ -250,8 +250,8 @@ io_export_recur_apoints(FILE *stream)
 {
 	recur_apoint_llist_node_t *i;
 	struct days_s *day;
-	char ical_datetime[MAX_LENGTH];
-	char ical_date[MAX_LENGTH];
+	char ical_datetime[BUFSIZ];
+	char ical_date[BUFSIZ];
 	
 	pthread_mutex_lock(&(recur_alist_p->mutex));
 	for (i = recur_alist_p->root; i != 0; i = i->next) { 
@@ -292,7 +292,7 @@ void
 io_export_apoints(FILE *stream)
 {
 	apoint_llist_node_t *i;
-	char ical_datetime[MAX_LENGTH];
+	char ical_datetime[BUFSIZ];
 	
 	pthread_mutex_lock(&(alist_p->mutex));
 	for (i = alist_p->root; i != 0; i = i->next) { 
@@ -333,21 +333,21 @@ io_init(char *cfile)
 {
 	FILE *data_file;
 	char *home;
-	char apts_file[MAX_LENGTH] = "";
+	char apts_file[BUFSIZ] = "";
 	int ch;
 
 	home = getenv("HOME");
 	if (home == NULL) {
 		home = ".";
 	}
-	snprintf(path_dir, MAX_LENGTH, "%s/" DIR_NAME, home);
-	snprintf(path_todo, MAX_LENGTH, "%s/" TODO_PATH, home);
-	snprintf(path_conf, MAX_LENGTH, "%s/" CONF_PATH, home);
+	snprintf(path_dir, BUFSIZ, "%s/" DIR_NAME, home);
+	snprintf(path_todo, BUFSIZ, "%s/" TODO_PATH, home);
+	snprintf(path_conf, BUFSIZ, "%s/" CONF_PATH, home);
 	if (cfile == NULL) {
-		snprintf(path_apts, MAX_LENGTH, "%s/" APTS_PATH, home);
+		snprintf(path_apts, BUFSIZ, "%s/" APTS_PATH, home);
 	} else {
-		snprintf(apts_file, MAX_LENGTH, "%s", cfile);
-		strncpy(path_apts, apts_file, MAX_LENGTH);
+		snprintf(apts_file, BUFSIZ, "%s", cfile);
+		strncpy(path_apts, apts_file, BUFSIZ);
 		/* check if the file exists, otherwise create it */
 		data_file = fopen(path_apts, "r");
 		if (data_file == NULL) {
@@ -402,7 +402,7 @@ io_save_cal(conf_t *conf, int layout)
 	struct event_s *k;
 	apoint_llist_node_t *j;
 	struct todo_s *i;
-	char theme_name[MAX_LENGTH];
+	char theme_name[BUFSIZ];
 	char *access_pb = _("Problems accessing data file ...");
 	char *config_txt =
 	    "#\n# Calcurse configuration file\n#\n# This file sets the configuration options used by Calcurse. These\n# options are usually set from within Calcurse. A line beginning with \n# a space or tab is considered to be a continuation of the previous line.\n# For a variable to be unset its value must be blank.\n# To set a variable to the empty string its value should be \"\".\n# Lines beginning with \"#\" are comments, and ignored by Calcurse.\n";
@@ -695,7 +695,7 @@ load_todo(void)
 	char *nl;
 	int nb_tod = 0;
 	int c, id;
-	char buf[MAX_LENGTH], e_todo[MAX_LENGTH];
+	char buf[BUFSIZ], e_todo[BUFSIZ];
 
 	data_file = fopen(path_todo, "r");
 	if (data_file == NULL) {
@@ -712,7 +712,7 @@ load_todo(void)
 			id = 9;
 			ungetc(c, data_file);
 		}
-		fgets(buf, MAX_LENGTH, data_file);
+		fgets(buf, BUFSIZ, data_file);
 		nl = strchr(buf, '\n');
 		if (nl) {
 			*nl = '\0';
diff --git a/src/recur.c b/src/recur.c
index 5f8467c..48d5db6 100755
--- a/src/recur.c
+++ b/src/recur.c
@@ -1,4 +1,4 @@
-/*	$calcurse: recur.c,v 1.23 2007/03/10 16:48:19 culot Exp $	*/
+/*	$calcurse: recur.c,v 1.24 2007/04/04 19:38:18 culot Exp $	*/
 
 /*
  * Calcurse - text-based organizer
@@ -26,6 +26,7 @@
 
 #include <ncurses.h>
 #include <stdlib.h>
+#include <stdio.h>
 #include <string.h>
 #include <ctype.h>
 #include <sys/types.h>
@@ -121,7 +122,9 @@ struct recur_event_s *recur_event_new(char *mesg, long day, int id,
  * Correspondance between the defines on recursive type, 
  * and the letter to be written in file. 
  */
-char recur_def2char(int define){
+char 
+recur_def2char(int define)
+{
 	char recur_char;
 
 	switch (define) {
@@ -138,7 +141,8 @@ char recur_def2char(int define){
 		recur_char = 'Y';
 		break;
 	}
-	return recur_char;
+
+	return (recur_char);
 }
 
 /* 
@@ -572,7 +576,7 @@ recur_repeat_item(int sel_year, int sel_month, int sel_day,
 	int ch = 0;
 	int valid_date = 0, date_entered = 0;
 	int year = 0, month = 0, day = 0;
-	char user_input[MAX_LENGTH] = "";
+	char user_input[BUFSIZ] = "";
 	char *mesg_type_1 = 
 	_("Enter the repetition type: (D)aily, (W)eekly, (M)onthly, (Y)early");
 	char *mesg_type_2 = _("[D/W/M/Y] ");
@@ -617,7 +621,7 @@ recur_repeat_item(int sel_year, int sel_month, int sel_day,
 
 	while (freq == 0) {
 		status_mesg(mesg_freq_1, "");
-		if (getstring(swin, user_input, MAX_LENGTH, 0, 1) == 
+		if (getstring(swin, user_input, BUFSIZ, 0, 1) == 
 		    GETSTRING_VALID) {
 			freq = atoi(user_input);
 			if (freq == 0) {
diff --git a/src/todo.c b/src/todo.c
index f1d4416..177f943 100755
--- a/src/todo.c
+++ b/src/todo.c
@@ -1,4 +1,4 @@
-/*	$calcurse: todo.c,v 1.10 2007/03/10 15:54:59 culot Exp $	*/
+/*	$calcurse: todo.c,v 1.11 2007/04/04 19:38:18 culot Exp $	*/
 
 /*
  * Calcurse - text-based organizer
@@ -43,10 +43,10 @@ todo_new_item(int total)
 	char *mesg = _("Enter the new ToDo item : ");
 	char *mesg_id = 
 		_("Enter the ToDo priority [1 (highest) - 9 (lowest)] :");
-	char todo_input[MAX_LENGTH] = "";
+	char todo_input[BUFSIZ] = "";
 
 	status_mesg(mesg, "");
-	if (getstring(swin, todo_input, MAX_LENGTH, 0, 1) == 
+	if (getstring(swin, todo_input, BUFSIZ, 0, 1) == 
 		GETSTRING_VALID) {
 		while ( (ch < '1') || (ch > '9') ) {
 			status_mesg(mesg_id, "");
@@ -149,7 +149,7 @@ int
 todo_chg_priority(int action, int item_num)
 {
 	struct todo_s *backup;
-	char backup_mesg[MAX_LENGTH];
+	char backup_mesg[BUFSIZ];
 	int backup_id;
 	int do_chg = 1, new_position;
 
diff --git a/src/vars.h b/src/vars.h
index f85526f..5a8415f 100755
--- a/src/vars.h
+++ b/src/vars.h
@@ -1,4 +1,4 @@
-/*	$calcurse: vars.h,v 1.8 2007/03/24 23:22:16 culot Exp $	*/
+/*	$calcurse: vars.h,v 1.9 2007/04/04 19:38:18 culot Exp $	*/
 
 /*
  * Calcurse - text-based organizer
@@ -30,17 +30,15 @@
 #include <ncurses.h> 
 #include <pthread.h>
 #include <stdbool.h>
+#include <stdio.h>
 
 #define DIR_NAME	".calcurse"
 #define TODO_PATH 	".calcurse/todo"
 #define APTS_PATH	".calcurse/apts"
 #define CONF_PATH	".calcurse/conf"
 
-#define MAX_LENGTH	512
-
 #define CTRL(x)         ((x) & 0x1f)
 #define ESCAPE		27
-
 #define ATTR_FALSE	0
 #define ATTR_TRUE	1
 #define ATTR_LOWEST	2
@@ -64,9 +62,9 @@ struct nbar_s {
 	int show;			/* display or hide the notify-bar */
 	int cntdwn;			/* warn when time left before next app
 					 * becomes lesser than cntdwn */  
-	char datefmt[MAX_LENGTH];	/* format for displaying date */
-	char timefmt[MAX_LENGTH];	/* format for displaying time */
-	char cmd[MAX_LENGTH];		/* notification command */
+	char datefmt[BUFSIZ];		/* format for displaying date */
+	char timefmt[BUFSIZ];		/* format for displaying time */
+	char cmd[BUFSIZ];		/* notification command */
 	char *shell;			/* user shell to launch notif. cmd */
 	pthread_mutex_t mutex;
 };
@@ -86,10 +84,10 @@ extern bool colorize;
 extern int days[12];
 extern char *monthnames[12];
 extern char *daynames[8];
-extern char path_dir[MAX_LENGTH];
-extern char path_todo[MAX_LENGTH];
-extern char path_apts[MAX_LENGTH];
-extern char path_conf[MAX_LENGTH];
+extern char path_dir[BUFSIZ];
+extern char path_todo[BUFSIZ];
+extern char path_apts[BUFSIZ];
+extern char path_conf[BUFSIZ];
 extern WINDOW *awin, *cwin, *twin, *swin;
 extern struct pad_s *apad;
 extern struct nbar_s *nbar;
-- 
cgit v1.2.3-70-g09d2