* 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

diff --git a/emufs_gui/gui.c b/emufs_gui/gui.c
index 79ea64841861da677ce330e4e5a2640daaa17beb..3800f9dc097d8ce8a5147c387e2f3fda040e0cfb 100644 (file)
--- 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 b146a5f62a844193baf88ad7f89b68bc636d9e02..a7c05b4b4df0f7f3d1b2c51d45930b899c62a67a 100644 (file)
--- 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]);