Uso un struct EMUFS_TIPO1_REG_HEADER para manejar más cómodo las cabeceras de

los registros.

diff --git a/emufs/tipo1.c b/emufs/tipo1.c
index 769d7b24cf47bc4505adc24cd5b6bc9738eb3c5b..4c68ce432a40e4e490fef30213682678f728d385 100644 (file)
--- a/emufs/tipo1.c
+++ b/emufs/tipo1.c
@@ -44,26 +44,32 @@
 #include <string.h>
 #include <unistd.h>
 
-/*------------------ Funciones privadas ----------------------*/
+/*------------------ Declaraciones privadas ----------------------*/
 
-int emufs_tipo1_header_jump(FILE*);
+/** Cabecera de un registro de un archivo tipo1. */
+typedef struct {
+       EMUFS_REG_ID   id;   /**< Identificador del registro. */
+       EMUFS_REG_SIZE size; /**< Tamaño del registro. */
+} EMUFS_TIPO1_REG_HEADER;
 
-size_t emufs_tipo1_header_size(void);
+static size_t emufs_tipo1_header_size(void);
 
-int emufs_tipo1_block_jump(EMUFS*, FILE*, EMUFS_BLOCK_ID);
+static int emufs_tipo1_header_jump(FILE*);
 
-void emufs_tipo1_escribir_reg_en_memoria(char* dst, EMUFS_REG_ID reg_id,
-               EMUFS_REG_SIZE reg_size, char* reg);
+static int emufs_tipo1_block_jump(EMUFS*, FILE*, EMUFS_BLOCK_ID);
+
+static void emufs_tipo1_escribir_reg_en_memoria(char*, EMUFS_TIPO1_REG_HEADER,
+               char*);
 
 /*------------------ Funciones públicas ----------------------*/
 
 int emufs_tipo1_inicializar(EMUFS* efs)
 {
        /* 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;
+       efs->leer_bloque       = emufs_tipo1_leer_bloque;
+       efs->grabar_registro   = emufs_tipo1_grabar_registro;
+       efs->borrar_registro   = emufs_tipo1_borrar_registro;
+       efs->leer_registro     = emufs_tipo1_leer_registro;
        efs->leer_registro_raw = emufs_tipo1_leer_registro_raw;
        return 0;
 }
@@ -76,8 +82,7 @@ void* emufs_tipo1_leer_registro(EMUFS* efs, EMUFS_REG_ID reg_id,
        EMUFS_BLOCK_ID block_id; /* id del bloque en donde esta el registro a leer */
        EMUFS_BLOCK_SIZE offset; /* offset del bloque leído */
        EMUFS_BLOCK_SIZE block_size; /* tamaño del bloque leído */
-       EMUFS_REG_SIZE curr_reg_size; /* tamaño del registro leído secuencialmente */
-       EMUFS_REG_ID curr_reg_id; /* id del registro leído secuencialmente */
+       EMUFS_TIPO1_REG_HEADER curr_reg_header; /* cabecera del registro a leer */
 
        block_id = emufs_idx_buscar_registro(efs, reg_id);
        if (block_id == EMUFS_NOT_FOUND) {
@@ -95,14 +100,11 @@ void* emufs_tipo1_leer_registro(EMUFS* efs, EMUFS_REG_ID reg_id,
        /* Busco secuencialmente en el bloque el registro a leer */
        offset = 0;
        do {
-               /* Copio el id del registro de la cabecera. */
-               memcpy(&curr_reg_id, block + offset, sizeof(EMUFS_REG_ID));
-               offset += sizeof(EMUFS_REG_ID);
-               /* Copio el tamaño del registro de la cabecera. */
-               memcpy(&curr_reg_size, block + offset, sizeof(EMUFS_REG_SIZE));
-               offset += sizeof(EMUFS_REG_SIZE);
-               if (curr_reg_id == reg_id) {
-                       registro = (char*) malloc(curr_reg_size);
+               /* Copio la cabecera del registro actual. */
+               memcpy(&curr_reg_header, block + offset, sizeof(EMUFS_TIPO1_REG_HEADER));
+               offset += sizeof(EMUFS_TIPO1_REG_HEADER);
+               if (curr_reg_header.id == reg_id) {
+                       registro = (char*) malloc(curr_reg_header.size);
                        if (registro == NULL) {
                                /* TODO Manejo de errores */
                                free(block);
@@ -110,12 +112,12 @@ void* emufs_tipo1_leer_registro(EMUFS* efs, EMUFS_REG_ID reg_id,
                                *err = 2; /* EMUFS_ERROR_OUT_OF_MEMORY */
                                return NULL;
                        }
-                       memcpy(registro, block + offset, curr_reg_size);
-                       *reg_size = curr_reg_size;
+                       memcpy(registro, block + offset, curr_reg_header.size);
+                       *reg_size = curr_reg_header.size;
                        break;
                }
                /* Desplazo el offset */
-               offset += curr_reg_size;
+               offset += curr_reg_header.size;
        } while (offset < block_size);
 
        free(block);
@@ -160,33 +162,35 @@ void* emufs_tipo1_leer_bloque(EMUFS* efs, EMUFS_BLOCK_ID block_id, int *err)
 EMUFS_REG_ID emufs_tipo1_grabar_registro(EMUFS* efs, void* reg,
                EMUFS_REG_SIZE reg_size, int* err)
 {
-       EMUFS_REG_ID   reg_id;
-       EMUFS_FREE     fs;
-       EMUFS_BLOCK_ID block_id;
-       char           name_f[255];
-       char*          block;
-       
+       EMUFS_TIPO1_REG_HEADER reg_header;
+       EMUFS_FREE             fs;
+       EMUFS_BLOCK_ID         block_id;
+       char                   name_f[255];
+       char*                  block;
+
        strcpy(name_f, efs->nombre);
        strcat(name_f, ".dat");
-       
+
+       /* pongo tamaño del registro en la cabecera. */
+       reg_header.size = reg_size;
        /* me devuelve el ID del bloque donde quepa un registro y el espacio libre en "fs"*/
-       block_id = emufs_fsc_buscar_lugar(efs, reg_size + sizeof(EMUFS_REG_ID)
-                       + sizeof(EMUFS_REG_SIZE), &fs);
+       block_id = emufs_fsc_buscar_lugar(efs, reg_size
+                       + sizeof(EMUFS_TIPO1_REG_HEADER), &fs);
        /* si no hay bloques con suficiente espacio creo un bloque nuevo */
        if (block_id == EMUFS_NOT_FOUND) {
                /* crear un nuevo bloque en memoria */
                block = (char*) malloc(efs->tam_bloque);
-               memset(block, 0, efs->tam_bloque);
                if (block == NULL) {
                        /* TODO Manejo de errores */
                        PERR("No hay memoria");
                        *err = 2; /* EMUFS_ERROR_OUT_OF_MEMORY */
                        return EMUFS_NOT_FOUND;
                }
-               /* graba el registro al principio del bloque */
-               reg_id = emufs_idx_get_new_id(efs, err);
+               memset(block, 0, efs->tam_bloque); /* inicializa bloque */
+               reg_header.id = emufs_idx_get_new_id(efs, err);
                /* graba registro en bloque */
-               emufs_tipo1_escribir_reg_en_memoria(block, reg_id, reg_size, reg);
+               /* TODO if (reg_size > efs->tam_bloque) */
+               emufs_tipo1_escribir_reg_en_memoria(block, reg_header, reg);
                /* graba el bloque en el archivo */
                block_id = emufs_tipo1_grabar_bloque(efs, block, block_id, err);
                if (*err) {
@@ -197,7 +201,7 @@ EMUFS_REG_ID emufs_tipo1_grabar_registro(EMUFS* efs, void* reg,
                free(block);
                /* grabo el nuevo registro en el archivo de espacios libres */
                *err = emufs_fsc_agregar(efs, block_id, efs->tam_bloque - reg_size
-                       - sizeof(EMUFS_REG_ID) - sizeof(EMUFS_REG_SIZE));
+                               - sizeof(EMUFS_TIPO1_REG_HEADER));
                if (*err) {
                        PERR("No se pudo agregar fsc");
                        return EMUFS_NOT_FOUND;
@@ -213,10 +217,10 @@ EMUFS_REG_ID emufs_tipo1_grabar_registro(EMUFS* efs, void* reg,
                }
                /* inserta el registro en el bloque */
                /* tengo que buscar un ID válido para el nuevo registro */
-               reg_id = emufs_idx_get_new_id(efs, err);
+               reg_header.id = emufs_idx_get_new_id(efs, err);
                /* graba registro en bloque */
                emufs_tipo1_escribir_reg_en_memoria(block + efs->tam_bloque - fs,
-                               reg_id, reg_size, reg);
+                               reg_header, reg);
                /* graba el bloque en el archivo */
                block_id = emufs_tipo1_grabar_bloque(efs, block, block_id, err);
                if (*err) {
@@ -227,7 +231,7 @@ EMUFS_REG_ID emufs_tipo1_grabar_registro(EMUFS* efs, void* reg,
                free(block);
                /* actualizo el archivo de espacios libres */
                *err = emufs_fsc_actualizar(efs, block_id, fs - reg_size
-                               - sizeof(EMUFS_REG_ID) - sizeof(EMUFS_REG_SIZE));
+                               - sizeof(EMUFS_TIPO1_REG_HEADER));
                if (*err) {
                        PERR("No se pudo actualizar fsc");
                        return EMUFS_NOT_FOUND;
@@ -235,13 +239,13 @@ EMUFS_REG_ID emufs_tipo1_grabar_registro(EMUFS* efs, void* reg,
        }
                
        /* actualizo el indice de bloques y registros */
-       *err = emufs_idx_agregar(efs, reg_id, block_id);
+       *err = emufs_idx_agregar(efs, reg_header.id, block_id);
        if (*err){
                PERR("No se pudo agregar idx");
                return EMUFS_NOT_FOUND;
        }
        
-       return reg_id;
+       return reg_header.id;
 }
 
 /*Graba un bloque en el archivo*/
@@ -310,6 +314,11 @@ int emufs_tipo1_borrar_registro(EMUFS *emu, EMUFS_REG_ID id_reg)
        return -1; /* FIXME Error */
 }
 
+size_t emufs_tipo1_header_size(void)
+{
+       return sizeof(EMUFS_Tipo) + sizeof(EMUFS_BLOCK_SIZE);
+}
+
 int emufs_tipo1_header_jump(FILE* fp)
 {
        if (fseek(fp, emufs_tipo1_header_size(), SEEK_CUR)) {
@@ -328,40 +337,31 @@ int emufs_tipo1_block_jump(EMUFS* efs, FILE* fp, EMUFS_BLOCK_ID block_count)
        return 0; /* EMUFS_OK */
 }
 
-size_t emufs_tipo1_header_size(void)
-{
-       return sizeof(EMUFS_Tipo) +      /* Cabecera de tipo de archivo */
-              sizeof(EMUFS_BLOCK_SIZE); /* Cabecera de tamaño del bloque */
-}
-
-void emufs_tipo1_escribir_reg_en_memoria(char* dst, EMUFS_REG_ID reg_id,
-               EMUFS_REG_SIZE reg_size, char* reg) {
-       /* grabo el id en el bloque */
-       memcpy(dst, &reg_id, sizeof(EMUFS_REG_ID));
-       /* incremento puntero de escritura */
-       dst += sizeof(EMUFS_REG_ID);
-       /* grabo el tamaño del registro en el bloque */
-       memcpy(dst, &reg_size, sizeof(EMUFS_REG_SIZE));
+void emufs_tipo1_escribir_reg_en_memoria(char* dst,    EMUFS_TIPO1_REG_HEADER header,
+               char* reg) {
+       /* grabo cabecera del registro en el bloque */
+       memcpy(dst, &header, sizeof(EMUFS_TIPO1_REG_HEADER));
        /* incremento puntero de escritura */
-       dst += sizeof(EMUFS_REG_SIZE);
+       dst += sizeof(EMUFS_TIPO1_REG_HEADER);
        /* grabo el registro en el bloque */
-       memcpy(dst, reg, reg_size);
+       memcpy(dst, reg, header.size);
 }
 
-EMUFS_REG_ID emufs_tipo1_modificar_registro(EMUFS *emu, EMUFS_REG_ID id, void *data, EMUFS_REG_SIZE size, int *error)
+EMUFS_REG_ID emufs_tipo1_modificar_registro(EMUFS *emu, EMUFS_REG_ID id,
+               void *data, EMUFS_REG_SIZE size, int *error)
 {
        emufs_tipo1_borrar_registro(emu, id);
        return emufs_tipo1_grabar_registro(emu, data, size, error);
 }
 
-void* emufs_tipo1_leer_registro_raw(EMUFS *efs, EMUFS_REG_ID id, EMUFS_REG_SIZE *size, int *pos)
+void* emufs_tipo1_leer_registro_raw(EMUFS *efs, EMUFS_REG_ID id,
+               EMUFS_REG_SIZE *size, int *pos)
 {
        char* block; /* bloque leido (en donde está el registro a leer) */
        EMUFS_BLOCK_ID block_id; /* id del bloque en donde esta el registro a leer */
        EMUFS_BLOCK_SIZE offset; /* offset del bloque leído */
        EMUFS_BLOCK_SIZE block_size; /* tamaño del bloque leído */
-       EMUFS_REG_SIZE curr_reg_size; /* tamaño del registro leído secuencialmente */
-       EMUFS_REG_ID curr_reg_id; /* id del registro leído secuencialmente */
+       EMUFS_TIPO1_REG_HEADER curr_reg_header; /* cabecera del registro a leer */
        int err;
 
        block_id = emufs_idx_buscar_registro(efs, id);
@@ -375,21 +375,18 @@ void* emufs_tipo1_leer_registro_raw(EMUFS *efs, EMUFS_REG_ID id, EMUFS_REG_SIZE
        /* Busco secuencialmente en el bloque el registro a leer */
        offset = 0;
        do {
-               /* Copio el id del registro de la cabecera. */
-               memcpy(&curr_reg_id, block + offset, sizeof(EMUFS_REG_ID));
-               offset += sizeof(EMUFS_REG_ID);
-               /* Copio el tamaño del registro de la cabecera. */
-               memcpy(&curr_reg_size, block + offset, sizeof(EMUFS_REG_SIZE));
-               offset += sizeof(EMUFS_REG_SIZE);
-               if (curr_reg_id == id) {
-                       *pos = offset-sizeof(EMUFS_REG_ID)-sizeof(EMUFS_REG_SIZE);
+               /* Copio la cabecera del registro. */
+               memcpy(&curr_reg_header, block + offset, sizeof(EMUFS_TIPO1_REG_HEADER));
+               offset += sizeof(EMUFS_TIPO1_REG_HEADER);
+               if (curr_reg_header.id == id) {
+                       *pos = offset - sizeof(EMUFS_TIPO1_REG_HEADER);
                        break;
                }
                /* Desplazo el offset */
-               offset += curr_reg_size;
+               offset += curr_reg_header.size;
        } while (offset < block_size);
 
-       (*size) = efs->tam_bloque;
+       (*size) = block_size;
        return block;
 }
 

diff --git a/emufs/tipo1.h b/emufs/tipo1.h
index 830e60ca361dcecba773b2a6fe46d88c6a5a4a3e..172248180318e76deed369687476865ade2be0df 100644 (file)
--- a/emufs/tipo1.h
+++ b/emufs/tipo1.h
@@ -40,25 +40,26 @@
 
 #include "emufs.h"
 
+/** Inicializa un EMUFS para poder ser utilizada como un archivo tipo1. */
 int emufs_tipo1_inicializar(EMUFS*);
 
-/** Lee el registro \param id_reg y lo almacena en \param ptr */
+/** Lee el registro \param id_reg y lo almacena en \c ptr. */
 void* emufs_tipo1_leer_registro(EMUFS*, EMUFS_REG_ID, EMUFS_REG_SIZE*, int*);
 
-/** Lee el bloque \param num_bloque y lo almacena en \param ptr */
+/** Lee el bloque \param num_bloque y lo almacena en \c ptr. */
 void* emufs_tipo1_leer_bloque(EMUFS*, EMUFS_BLOCK_ID, int*);
 
-/** Graba el registro apuntado por \param ptr en el archivo */
+/** Graba el registro apuntado por \c ptr en el archivo. */
 EMUFS_REG_ID emufs_tipo1_grabar_registro(EMUFS*, void*, EMUFS_REG_SIZE, int*);
 
-/** Graba el bloque apuntado por \param ptr en el archivo */
+/** Graba el bloque apuntado por \c ptr en el archivo. */
 EMUFS_BLOCK_ID emufs_tipo1_grabar_bloque(EMUFS*, void*, EMUFS_BLOCK_ID, int*);
 
 int emufs_tipo1_buscar_registro(EMUFS*, EMUFS_REG_ID);
 
 int emufs_tipo1_borrar_registro(EMUFS*, EMUFS_REG_ID);
 
-/** Método para modificar un registro */
+/** Método para modificar un registro. */
 EMUFS_REG_ID emufs_tipo1_modificar_registro(EMUFS *emu, EMUFS_REG_ID, void*, EMUFS_REG_SIZE, int*);
 
 void* emufs_tipo1_leer_registro_raw(EMUFS *emu, EMUFS_REG_ID id, EMUFS_REG_SIZE *size, int *pos);