From c44140901674eea943d44ef522337fae17feb3c1 Mon Sep 17 00:00:00 2001
From: Ricardo Markiewicz <gazer.arg@gmail.com>
Date: Wed, 14 Apr 2004 07:28:48 +0000
Subject: [PATCH 1/1]  * Agrego un msg de advertencia a la GUI si se compila
 con -DDEBUG, para prevenir  al usuario de redirigir el flujo stderr a un
 archivo por posibles problemas  entre curses y fprintf  * Mejoro parser de
 opciones

---
 emufs_gui/gui.c       | 62 ++++++++++++++++++++++++++++++++-----------
 emufs_gui/registros.c |  1 -
 2 files changed, 46 insertions(+), 17 deletions(-)

diff --git a/emufs_gui/gui.c b/emufs_gui/gui.c
index 79ea648..3800f9d 100644
--- a/emufs_gui/gui.c
+++ b/emufs_gui/gui.c
@@ -22,23 +22,28 @@ void menu_articulos();
 WINDOW *msg_box(WINDOW *win, int w, int h, const char *format, ...);
 void msg_box_free(WINDOW *padre, WINDOW *win);
 
+typedef enum {PARAM_OK, SHOW_HELP, TIPO_NO_DEFINIDO, TIPO_INVALIDO} t_Param;
+
 /* Verifica Argumentos */
-int param_ok(int argc, char *argv[])
+t_Param param_ok(int argc, char *argv[])
 {
-	int n;
-	switch (argc) {
-		case 1:
-			return 1;
-		case 2:
-			if (strcmp("-h", argv[1]) == 0) {
-				return 0;
+	int n,i;
+	for(i=1; i<argc; i++) {
+		if ((strcmp(argv[i], "-h")==0) || (strcmp(argv[i], "--help")==0)) return SHOW_HELP;
+
+		if (strcmp(argv[i]+strlen(argv[i])-3, "xml") == 0) {
+			/* Luego del archivo XML debe seguir el tipo */
+			if ((i+1)<argc) {
+				n = atoi(argv[i+1]);
+				if ((n < 1) || (n > 3)) return TIPO_INVALIDO;
+			} else {
+				/* Ops, no hay mas parametros */
+				return TIPO_NO_DEFINIDO;
 			}
-		case 3:
-			n = atoi(argv[2]);
-			if ((n<1) || (n>3))
-				return 0;
+		}
 	}
-	return 1;
+
+	return PARAM_OK;
 }
 
 void print_help(char *s)
@@ -54,11 +59,36 @@ int main(int argc, char *argv[])
 	int c, fin=0;
 	WINDOW *dialog;
 
-	if (!param_ok(argc, argv)) {
-		print_help(argv[0]);
-		return -1;
+	switch (param_ok(argc, argv)) {
+		case SHOW_HELP:
+			print_help(argv[0]);
+			return 0;
+		case TIPO_NO_DEFINIDO:
+			printf("Falta parámetro requerido. Luego del nombre del archivo debe especificar el tipo de archivo\n");
+			return 1;
+		case TIPO_INVALIDO:
+			printf("Tipo de archivo no valido. Los valores posibles para el tipo de archivo son:\n");
+			printf("\t1 - Archivo de bloque parametrizado y registro de long. variable.\n");
+			printf("\t2 - Archivo de registros variables sin bloques.\n");
+			printf("\t3 - Archivos de bloque parametrizado y registro de long. parametrizada.\n");
+			return 2;
+		case PARAM_OK:
+			fin = 0;
 	}
 
+#ifdef DEBUG
+	printf("CUIDADO! - Uds esta a punto de ejecutar EMUFS Gui compilado con mensajes de debug (-DDEBUG). ");
+	printf("Esto puede causar que ante un error alguna función trate de emitir un mensaje por pantalla ");
+	printf("haciendo que el aspecto visual se vea desvirtuado.\n\n");
+	printf("Todos los mensajes de error se envian por stderr, por lo que es conveniente que vuelva a ejecutar ");
+	printf("el programa de la siguiente manera :\n");
+	printf("\t#> %s <parametros> 2> error.log\n\n", argv[0]);
+	printf("De esta forma el SO se encargaga de redirigir stderr al archivo error.log y evitar algun problema en ");
+	printf("visualizacion de la aplicacion.\n");
+	printf("Para continuar **bajo su propio riesgo** presione una tecla. Puede cancelar la ejecucion en este punto con CTRL+C\n");
+	fgetc(stdin);
+#endif
+
 	/* Inicio Curses */
 	signal(SIGINT, finish);
 	initscr();
diff --git a/emufs_gui/registros.c b/emufs_gui/registros.c
index b146a5f..a7c05b4 100644
--- a/emufs_gui/registros.c
+++ b/emufs_gui/registros.c
@@ -91,7 +91,6 @@ void ver_registros(WINDOW *padre, int w, int h)
 		switch (c) {
 			case 'e': /* Quiero editar !!! */
 				sprintf(codigo, "%lu", ant_indice+1);
-				fprintf(stderr, "%s", codigo);
 				art_modificar(codigo);	
 				/* Vuelvo a cargar el articulo actual */
 				free(data[ACT]);
-- 
2.43.0