]> git.llucax.com Git - z.facultad/75.06/emufs.git/blobdiff - emufs/tipo1.c
Se pasan algunas funciones "privadas" al .c.
[z.facultad/75.06/emufs.git] / emufs / tipo1.c
index 024839bcf10521d264d985f8c951b91091921085..5d8dfffbc34f176efd2e8eb564565bfc4c660dda 100644 (file)
@@ -1,4 +1,4 @@
-/* vim: set noexpandtab tabstop=4 shiftwidth=4:
+/* vim: set noexpandtab tabstop=4 shiftwidth=4 wrap:
  *----------------------------------------------------------------------------
  *                                  emufs
  *----------------------------------------------------------------------------
  *----------------------------------------------------------------------------
  *                                  emufs
  *----------------------------------------------------------------------------
 #include <string.h>
 #include <unistd.h>
 
 #include <string.h>
 #include <unistd.h>
 
-/*------------------ Funciones privadas ----------------------*/
+#ifndef MIN
+#      define MIN(x, y) (((x) > (y)) ? (y) : (x))
+#endif
 
 
-int emufs_tipo1_header_jump(FILE*);
+/*------------------ Declaraciones privadas ----------------------*/
 
 
-size_t emufs_tipo1_header_size(void);
+/** 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;
 
 
-int emufs_tipo1_block_jump(EMUFS*, FILE*, EMUFS_BLOCK_ID);
+static size_t emufs_tipo1_header_size(void);
 
 
-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_header_jump(FILE*);
+
+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*);
+
+static void emufs_tipo1_escribir_reg_chunk_en_memoria(char* dst,
+               EMUFS_TIPO1_REG_HEADER header, char* reg, EMUFS_REG_SIZE reg_size);
+
+/** Lee el bloque \param num_bloque y lo almacena en \c ptr. */
+static void* emufs_tipo1_leer_bloque(EMUFS*, EMUFS_BLOCK_ID, int*);
+
+/** Graba el bloque apuntado por \c ptr en el archivo. */
+static EMUFS_BLOCK_ID emufs_tipo1_grabar_bloque(EMUFS*, void*, EMUFS_BLOCK_ID,
+               int*);
 
 /*------------------ Funciones públicas ----------------------*/
 
 int emufs_tipo1_inicializar(EMUFS* efs)
 {
        /* Asigna punteros a funciones. */
 
 /*------------------ 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;
 }
 
        return 0;
 }
 
@@ -74,9 +95,7 @@ void* emufs_tipo1_leer_registro(EMUFS* efs, EMUFS_REG_ID reg_id,
        char* registro; /* 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 */
        char* registro; /* 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 */
 
        block_id = emufs_idx_buscar_registro(efs, reg_id);
        if (block_id == EMUFS_NOT_FOUND) {
 
        block_id = emufs_idx_buscar_registro(efs, reg_id);
        if (block_id == EMUFS_NOT_FOUND) {
@@ -87,21 +106,28 @@ void* emufs_tipo1_leer_registro(EMUFS* efs, EMUFS_REG_ID reg_id,
        }
        if (!(block = (char*) emufs_tipo1_leer_bloque(efs, block_id, err))) {
                /* TODO Manejo de errores */
        }
        if (!(block = (char*) emufs_tipo1_leer_bloque(efs, block_id, err))) {
                /* TODO Manejo de errores */
-               PERR("no se pudo leer el bloque");
+               PERR("no se pudo reservar memoria");
+               *err = 2; /* EMUFS_ERROR_OUT_OF_MEMORY */
                return NULL;
        }
 
        /* Busco secuencialmente en el bloque el registro a leer */
        offset = 0;
        do {
                return NULL;
        }
 
        /* 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) {
+                       /* tamaño máximo ultilizable para datos en un bloque */
+                       EMUFS_BLOCK_SIZE block_space
+                                       = efs->tam_bloque - sizeof(EMUFS_TIPO1_REG_HEADER);
+                       /* tamaño de la porción de registro que se guarda */
+                       EMUFS_REG_SIZE chunk_size = 0; 
+                       /* puntero a la porción actual del registro */
+                       char* chunk_ptr;
+
+                       *reg_size = curr_reg_header.size;
+                       registro = chunk_ptr = (char*) malloc(*reg_size);
                        if (registro == NULL) {
                                /* TODO Manejo de errores */
                                free(block);
                        if (registro == NULL) {
                                /* TODO Manejo de errores */
                                free(block);
@@ -109,18 +135,76 @@ void* emufs_tipo1_leer_registro(EMUFS* efs, EMUFS_REG_ID reg_id,
                                *err = 2; /* EMUFS_ERROR_OUT_OF_MEMORY */
                                return NULL;
                        }
                                *err = 2; /* EMUFS_ERROR_OUT_OF_MEMORY */
                                return NULL;
                        }
-                       memcpy(registro, block + offset, curr_reg_size);
-                       *reg_size = curr_reg_size;
+                       while (1) {
+                               chunk_ptr += chunk_size; /* Avanzo para guardar prox chunk */
+                               curr_reg_header.size -= chunk_size; /* Resto lo que ya guardé */
+                               chunk_size = MIN(curr_reg_header.size, block_space);
+                               /* copio porción de registro en el buffer */
+                               memcpy(chunk_ptr, block + offset, chunk_size);
+                                /* falta leer un bloque */
+                               if (curr_reg_header.size > block_space) {
+                                       free(block);
+                                       if (!(block = (char*) emufs_tipo1_leer_bloque(efs,
+                                                                       ++block_id, err))) {
+                                               /* TODO Manejo de errores */
+                                               free(registro);
+                                               PERR("no se pudo reservar memoria");
+                                               *err = 2; /* EMUFS_ERROR_OUT_OF_MEMORY */
+                                               return NULL;
+                                       }
+                               } else { /* se terminó de leer */
+                                       break;
+                               }
+                       }
                        break;
                }
                /* Desplazo el offset */
                        break;
                }
                /* Desplazo el offset */
-               offset += curr_reg_size;
-       } while (offset < block_size);
+               offset += curr_reg_header.size;
+
+       /* esto no debería ser nunca false porque sé positivamente que el */
+       } while (offset < efs->tam_bloque); /* registro está en el bloque */
 
        free(block);
        return registro;
 }
 
 
        free(block);
        return registro;
 }
 
+/* @todo TODO hacer que soporte registros de más de un bloque */
+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_TIPO1_REG_HEADER curr_reg_header; /* cabecera del registro a leer */
+       int err;
+
+       block_id = emufs_idx_buscar_registro(efs, id);
+       if (block_id == EMUFS_NOT_FOUND) {
+               return NULL;
+       }
+       err = 0;
+       if (!(block = (char*) emufs_tipo1_leer_bloque(efs, block_id, &err))) {
+               return NULL;
+       }
+
+       /* Busco secuencialmente en el bloque el registro a leer */
+       offset = 0;
+       do {
+               /* 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_header.size;
+       } while (offset < efs->tam_bloque);
+
+       (*size) = efs->tam_bloque;
+       return block;
+}
+
 void* emufs_tipo1_leer_bloque(EMUFS* efs, EMUFS_BLOCK_ID block_id, int *err)
 {
        FILE* file;
 void* emufs_tipo1_leer_bloque(EMUFS* efs, EMUFS_BLOCK_ID block_id, int *err)
 {
        FILE* file;
@@ -156,23 +240,33 @@ void* emufs_tipo1_leer_bloque(EMUFS* efs, EMUFS_BLOCK_ID block_id, int *err)
        return block;
 }
 
        return block;
 }
 
-EMUFS_REG_ID emufs_tipo1_grabar_registro(EMUFS* efs, void* reg,
-               EMUFS_REG_SIZE reg_size, 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; /* cabecera del registro a guardar */
+       EMUFS_FREE             fs; /* espacio libre en el bloque */
+       EMUFS_BLOCK_ID         block_id; /* identificador del 1er bloque */
+       char*                  block; /* buffer del bloque a guardar en disco */
+       char                   name_f[255];
+
        strcpy(name_f, efs->nombre);
        strcat(name_f, ".dat");
        strcpy(name_f, efs->nombre);
        strcat(name_f, ".dat");
-       
-       /* 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);
+
+       /* pongo tamaño del registro en la cabecera. */
+       reg_header.size = reg_size;
+       /* busco lugar para el registro en un bloque existente */
+       block_id = emufs_fsc_buscar_lugar(efs, sizeof(EMUFS_TIPO1_REG_HEADER)
+                       + reg_size, &fs);
        /* si no hay bloques con suficiente espacio creo un bloque nuevo */
        if (block_id == EMUFS_NOT_FOUND) {
        /* si no hay bloques con suficiente espacio creo un bloque nuevo */
        if (block_id == EMUFS_NOT_FOUND) {
+               /* tamaño máximo ultilizable para datos en un bloque */
+               EMUFS_BLOCK_SIZE block_space = efs->tam_bloque - sizeof(EMUFS_TIPO1_REG_HEADER);
+               /* identificador del bloque que se guarda */
+               EMUFS_BLOCK_ID curr_block_id = EMUFS_NOT_FOUND;
+               /* tamaño de la porción de registro que se guarda */
+               EMUFS_REG_SIZE chunk_size = 0; 
+               /* puntero a la poción del registro */
+               char* chunk_ptr = reg; 
+
                /* crear un nuevo bloque en memoria */
                block = (char*) malloc(efs->tam_bloque);
                if (block == NULL) {
                /* crear un nuevo bloque en memoria */
                block = (char*) malloc(efs->tam_bloque);
                if (block == NULL) {
@@ -181,25 +275,35 @@ EMUFS_REG_ID emufs_tipo1_grabar_registro(EMUFS* efs, void* reg,
                        *err = 2; /* EMUFS_ERROR_OUT_OF_MEMORY */
                        return EMUFS_NOT_FOUND;
                }
                        *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);
-               /* graba registro en bloque */
-               emufs_tipo1_escribir_reg_en_memoria(block, reg_id, reg_size, reg);
-               /* graba el bloque en el archivo */
-               block_id = emufs_tipo1_grabar_bloque(efs, block, block_id, err);
-               if (*err) {
-                       PERR("error al grabar bloque");
-                       free(block);
-                       return EMUFS_NOT_FOUND;
-               }
+               reg_header.id = emufs_idx_get_new_id(efs, err);
+               do {
+                       memset(block, 0, efs->tam_bloque); /* inicializa bloque */
+                       chunk_ptr += chunk_size; /* Avanzo para guardar prox chunk */
+                       reg_header.size -= chunk_size; /* Resto lo que ya guardé */
+                       chunk_size = MIN(reg_header.size, block_space);
+                       /* graba porción del registro en bloque */
+                       emufs_tipo1_escribir_reg_chunk_en_memoria(block, reg_header, chunk_ptr, chunk_size);
+                       /* graba el bloque en el archivo */
+                       curr_block_id = emufs_tipo1_grabar_bloque(efs, block, EMUFS_NOT_FOUND, err);
+                       if (*err) {
+                               PERR("error al grabar bloque");
+                               free(block);
+                               return EMUFS_NOT_FOUND;
+                       }
+                       /* grabo el nuevo registro en el archivo de espacios libres */
+                       *err = emufs_fsc_agregar(efs, curr_block_id, block_space - chunk_size);
+                       if (*err) {
+                               PERR("No se pudo agregar fsc");
+                               free(block);
+                               return EMUFS_NOT_FOUND;
+                       }
+                       /* si es el primer id de bloque obtenido, lo guardo para
+                        * agregarlo después al archivo de índices. */
+                       if (block_id == EMUFS_NOT_FOUND) {
+                               block_id = curr_block_id;
+                       }
+               } while (reg_header.size > block_space);
                free(block);
                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));
-               if (*err) {
-                       PERR("No se pudo agregar fsc");
-                       return EMUFS_NOT_FOUND;
-               }
 
        /* Encontró espacio en un bloque existente, graba registro ahí */
        } else {
 
        /* Encontró espacio en un bloque existente, graba registro ahí */
        } else {
@@ -211,10 +315,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 */
                }
                /* 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,
                /* 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) {
                /* graba el bloque en el archivo */
                block_id = emufs_tipo1_grabar_bloque(efs, block, block_id, err);
                if (*err) {
@@ -225,7 +329,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
                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;
                if (*err) {
                        PERR("No se pudo actualizar fsc");
                        return EMUFS_NOT_FOUND;
@@ -233,16 +337,125 @@ EMUFS_REG_ID emufs_tipo1_grabar_registro(EMUFS* efs, void* reg,
        }
                
        /* actualizo el indice de bloques y registros */
        }
                
        /* 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;
        }
        
        if (*err){
                PERR("No se pudo agregar idx");
                return EMUFS_NOT_FOUND;
        }
        
-       return reg_id;
+       return reg_header.id;
+}
+
+int emufs_tipo1_borrar_registro(EMUFS* efs, EMUFS_REG_ID reg_id)
+{
+       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_TIPO1_REG_HEADER curr_reg_header; /* cabecera del registro a leer */
+       int err = 0; /* para almacenar código de error */
+
+       block_id = emufs_idx_buscar_registro(efs, reg_id);
+       if (block_id == EMUFS_NOT_FOUND) {
+               /* TODO Manejo de errores */
+               PERR("Registro no encontrado");
+               return EMUFS_NOT_FOUND;
+       }
+       if (!(block = (char*) emufs_tipo1_leer_bloque(efs, block_id, &err))) {
+               /* TODO Manejo de errores */
+               PERR("no se pudo reservar memoria");
+               return err;
+       }
+
+       /* Busco secuencialmente en el bloque el registro a leer */
+       offset = 0;
+       do {
+               /* Copio la cabecera del registro actual. */
+               memcpy(&curr_reg_header, block + offset, sizeof(EMUFS_TIPO1_REG_HEADER));
+               if (curr_reg_header.id == reg_id) {
+                       /* identificador del bloque actual */
+                       EMUFS_BLOCK_ID curr_block_id = block_id;
+                       /* tamaño máximo ultilizable para datos en un bloque */
+                       EMUFS_BLOCK_SIZE block_space
+                                       = efs->tam_bloque - sizeof(EMUFS_TIPO1_REG_HEADER);
+                       EMUFS_FREE fs; /* cantidad de espacio libre en el bloque */
+
+                       while (1) {
+                               /* actualizo archivo de espacio libre por bloque */
+                               fs = emufs_fsc_get_fs(efs, curr_block_id)
+                                       + MIN(curr_reg_header.size, block_space)
+                                       + sizeof(EMUFS_TIPO1_REG_HEADER);
+                               if ((err = emufs_fsc_actualizar(efs, curr_block_id, fs))) {
+                                       /* TODO Manejo de errores */
+                                       PERR("no se pudo actualizar .fsc");
+                                       free(block);
+                                       return err;
+                               }
+                               /* falta liberar un bloque (o porción) */
+                               if (curr_reg_header.size > block_space) {
+                                       free(block);
+                                       if (!(block = (char*) emufs_tipo1_leer_bloque(efs,
+                                                                       ++curr_block_id, &err))) {
+                                               /* TODO Manejo de errores */
+                                               PERR("no se pudo leer el bloque");
+                                               return err;
+                                       }
+                                       /* copio la cabecera del primer registro (si ocupa más de un
+                                        * registro está en bloques contiguos) */
+                                       memcpy(&curr_reg_header, block,
+                                                       sizeof(EMUFS_TIPO1_REG_HEADER));
+                               } else { /* se terminó de leer */
+                                       break;
+                               }
+                       }
+
+                       /* actualizo archivo de identificadores de registros borrados */
+                       if ((err = emufs_did_agregar(efs, reg_id))) {
+                               /* TODO Manejo de errores */
+                               PERR("no se pudo actualizar .did");
+                               free(block);
+                               return err;
+                       }
+                       /*actualizo archivo .idx*/
+                       if ((err = emufs_idx_borrar(efs, reg_id))) {
+                               /* TODO Manejo de errores */
+                               PERR("no se pudo actualizar .did");
+                               free(block);
+                               return err;
+                       }
+
+                       /* desplazo registros a izquierda */
+                       {   /* offset del fin del registro a borrar */
+                               EMUFS_BLOCK_SIZE offset_reg_end = offset
+                                       + sizeof(EMUFS_TIPO1_REG_HEADER) + curr_reg_header.size;
+                               /* si es necesario desplazar */
+                               if (offset < offset_reg_end) {
+                                       /* muevo la porción de bloque a izquierda */
+                                       memcpy(block + offset, block + offset_reg_end,
+                                               efs->tam_bloque - offset_reg_end);
+                               }
+                       }
+                       /* guardo el bloque en disco */
+                       emufs_tipo1_grabar_bloque(efs, block, curr_block_id, &err);
+                       if (err) {
+                               /* TODO Manejo de errores */
+                               PERR("no se pudo grabar bloque en disco");
+                               free(block);
+                               return err;
+                       }
+
+                       break; /* salgo del loop, ya hice todo lo que tenía que hacer */
+               }
+               /* desplazo el offset */
+               offset += sizeof(EMUFS_TIPO1_REG_HEADER) + curr_reg_header.size;
+
+       /* esto no debería ser nunca false porque sé positivamente que el */
+       } while (offset < efs->tam_bloque); /* registro está en el bloque */
+
+       free(block);
+       return 0; /* EMUFS_OK */
 }
 
 }
 
-/*Graba un bloque en el archivo*/
+/** Graba un bloque en el archivo. */
 EMUFS_BLOCK_ID emufs_tipo1_grabar_bloque(EMUFS *efs, void *block,
                EMUFS_BLOCK_ID block_id, int* err)
 {
 EMUFS_BLOCK_ID emufs_tipo1_grabar_bloque(EMUFS *efs, void *block,
                EMUFS_BLOCK_ID block_id, int* err)
 {
@@ -297,15 +510,16 @@ EMUFS_BLOCK_ID emufs_tipo1_grabar_bloque(EMUFS *efs, void *block,
        return block_id;
 }
 
        return block_id;
 }
 
-/*borra un registro de un bloque y acomoda los registros que quedan*/
-int emufs_tipo1_buscar_registro(EMUFS *emu, EMUFS_REG_ID id_reg)
+EMUFS_REG_ID emufs_tipo1_modificar_registro(EMUFS *emu, EMUFS_REG_ID id,
+               void *data, EMUFS_REG_SIZE size, int *error)
 {
 {
-       return -1; /* FIXME Error */
+       emufs_tipo1_borrar_registro(emu, id);
+       return emufs_tipo1_grabar_registro(emu, data, size, error);
 }
 
 }
 
-int emufs_tipo1_borrar_registro(EMUFS *emu, EMUFS_REG_ID id_reg)
+size_t emufs_tipo1_header_size(void)
 {
 {
-       return -1; /* FIXME Error */
+       return sizeof(EMUFS_Tipo) + sizeof(EMUFS_BLOCK_SIZE);
 }
 
 int emufs_tipo1_header_jump(FILE* fp)
 }
 
 int emufs_tipo1_header_jump(FILE* fp)
@@ -326,28 +540,20 @@ int emufs_tipo1_block_jump(EMUFS* efs, FILE* fp, EMUFS_BLOCK_ID block_count)
        return 0; /* EMUFS_OK */
 }
 
        return 0; /* EMUFS_OK */
 }
 
-size_t emufs_tipo1_header_size(void)
+void emufs_tipo1_escribir_reg_en_memoria(char* dst, EMUFS_TIPO1_REG_HEADER header,
+               char* reg)
 {
 {
-       return sizeof(EMUFS_Tipo) +      /* Cabecera de tipo de archivo */
-              sizeof(EMUFS_BLOCK_SIZE); /* Cabecera de tamaño del bloque */
+       emufs_tipo1_escribir_reg_chunk_en_memoria(dst, header, reg, header.size);
 }
 
 }
 
-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_chunk_en_memoria(char* dst,
+               EMUFS_TIPO1_REG_HEADER header, char* reg, EMUFS_REG_SIZE reg_size)
+{
+       /* grabo cabecera del registro en el bloque */
+       memcpy(dst, &header, sizeof(EMUFS_TIPO1_REG_HEADER));
        /* incremento puntero de escritura */
        /* 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);
 }
 
        /* grabo el registro en el bloque */
        memcpy(dst, reg, reg_size);
 }
 
-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);
-}