Se mueve Doxyfile a la raiz para ejecutarlo mas facil.

[z.facultad/75.06/emufs.git] / emufs / emufs.c

diff --git a/emufs/emufs.c b/emufs/emufs.c
index a6a774c32dff5cb6aff9661a808a1075b2256ca9..7ddd4bd72c608f3a903187a02f532cf7b648d337 100644 (file)
--- a/emufs/emufs.c
+++ b/emufs/emufs.c
@@ -79,22 +79,26 @@ int emufs_crear_archivo_auxiliar(const char* name, const char* ext)
 }
 
 
-EMUFS *emufs_crear(const char *filename, char tipo, unsigned long tam_bloque, unsigned long tam_reg)
+EMUFS *emufs_crear(const char *filename, EMUFS_TYPE tipo,
+               EMUFS_BLOCK_SIZE tam_bloque, EMUFS_REG_SIZE tam_reg)
 {
        char name[255];
        FILE *fp;
        EMUFS *efs;
 
-       /* Si no es un tipo conocido, sale */
+       /* Si no es un tipo conocido, sale. */
        if ((tipo != T1) && (tipo != T2) && (tipo != T3)) {
                return NULL;
        }
 
        /* Inicializa parámetros comunes. */
        efs = (EMUFS*) malloc(sizeof(EMUFS));
-       efs->tipo = tipo;
+       if (efs == NULL) {
+               return NULL;
+       }
+       efs->tipo       = tipo;
        efs->tam_bloque = tam_bloque;
-       efs->nombre = str_dup(filename);
+       efs->nombre     = str_dup(filename);
 
        /* Abre archivo de datos. */
        strcpy(name, filename);
@@ -108,7 +112,7 @@ EMUFS *emufs_crear(const char *filename, char tipo, unsigned long tam_bloque, un
        }
 
        /* Guarda cabecera común. */
-       fwrite(&tipo, sizeof(char), 1, fp); /* FIXME no debería ser sizeof(EMUFS_TYPE) ? */
+       fwrite(&tipo, sizeof(EMUFS_TYPE), 1, fp);
 
        /* Crea archivo de índice. */
        if (emufs_idx_crear(efs)) {
@@ -138,14 +142,10 @@ EMUFS *emufs_crear(const char *filename, char tipo, unsigned long tam_bloque, un
        switch (tipo) {
 
                case T1:
-                       /* Asigna punteros a funciones. */
-                       efs->leer_bloque     = emufs_tipo1_leer_bloque;
-                       efs->leer_registro   = emufs_tipo1_leer_registro;
-                       efs->grabar_registro = emufs_tipo1_grabar_registro;
-                       efs->borrar_registro = emufs_tipo1_borrar_registro;
+                       emufs_tipo1_inicializar(efs);
 
                        /* Guarda cabeceras propias. */
-                       fwrite(&tam_bloque, sizeof(unsigned long), 1, fp);
+                       fwrite(&tam_bloque, sizeof(EMUFS_BLOCK_SIZE), 1, fp);
 
                        break;
 
@@ -160,8 +160,8 @@ EMUFS *emufs_crear(const char *filename, char tipo, unsigned long tam_bloque, un
                        efs->borrar_registro = emufs_tipo3_borrar_registro;
 
                        /* Guarda cabeceras propias. */
-                       fwrite(&tam_bloque, sizeof(unsigned int), 1, fp);
-                       fwrite(&tam_reg, sizeof(unsigned int), 1, fp);
+                       fwrite(&tam_bloque, sizeof(EMUFS_BLOCK_SIZE), 1, fp);
+                       fwrite(&tam_reg, sizeof(EMUFS_REG_SIZE), 1, fp);
                        
                        break;
 
@@ -173,7 +173,7 @@ EMUFS *emufs_crear(const char *filename, char tipo, unsigned long tam_bloque, un
 
 EMUFS *emufs_abrir(const char *filename)
 {
-       EMUFS *tmp;
+       EMUFS *efs;
        char name[255];
        char tipo;
        FILE *fp;
@@ -184,35 +184,53 @@ EMUFS *emufs_abrir(const char *filename)
        /* Trato de determinar el tipo de archivo */
        fp = fopen(name, "r");
        if (fp == NULL) return NULL;
-       fread(&tipo, sizeof(char), 1, fp);
-       if ((tipo < 0) || (tipo > 2)) {
+       fread(&tipo, sizeof(EMUFS_TYPE), 1, fp);
+
+       /* Si no es un tipo conocido, sale. */
+       if ((tipo != T1) && (tipo != T2) && (tipo != T3)) {
                fclose(fp);
                return NULL;
        }
        
-       tmp = (EMUFS *)malloc(sizeof(EMUFS));
-       if (tmp == NULL) {
+       /* Inicializa parámetros comunes. */
+       efs = (EMUFS*) malloc(sizeof(EMUFS));
+       if (efs == NULL) {
                fclose(fp);
                return NULL;
        }
+       efs->tipo = tipo;
+       efs->nombre = str_dup(filename);
 
        switch (tipo) {
                case T1:
-               break;
+                       emufs_tipo1_inicializar(efs);
+                       /* Lee cabeceras propias. */
+                       if (!fread(&(efs->tam_bloque), sizeof(EMUFS_BLOCK_SIZE), 1, fp)) {
+                               free(efs->nombre);
+                               free(efs);
+                               fclose(fp);
+                               return NULL;
+                       }
+                       break;
                case T2:
-               break;
+                       break;
                case T3:
-                       tmp->tipo = tipo;
-                       fread(&tmp->tam_bloque, sizeof(int), 1, fp);
-                       tmp->leer_bloque = emufs_tipo3_leer_bloque;
-                       tmp->leer_registro = emufs_tipo3_leer_registro;
-                       tmp->grabar_registro = emufs_tipo3_grabar_registro;
-                       tmp->borrar_registro = emufs_tipo3_borrar_registro;
-                       tmp->nombre = str_dup(filename);
+                       if (!fread(&(efs->tam_bloque), sizeof(EMUFS_BLOCK_SIZE), 1, fp)) {
+                               free(efs->nombre);
+                               free(efs);
+                               fclose(fp);
+                               return NULL;
+                       }
+                       /* FIXME no falta leer el tamaño del registro????  */
+                       efs->leer_bloque = emufs_tipo3_leer_bloque;
+                       efs->leer_registro = emufs_tipo3_leer_registro;
+                       efs->grabar_registro = emufs_tipo3_grabar_registro;
+                       efs->borrar_registro = emufs_tipo3_borrar_registro;
+                       break;
        }
 
        fclose(fp);
-       return tmp;
+       return efs;
 }
 
 int emufs_destruir(EMUFS *e)