From 9e708c77b6a6ea06b945986b9b2dd93b15c374d7 Mon Sep 17 00:00:00 2001
From: Leandro Lucarella <llucax@gmail.com>
Date: Fri, 16 Apr 2004 06:53:18 +0000
Subject: [PATCH 1/1] Se generaliza el algoritmo de leer_registro(). Ahora, al
 igual que grabar_registro() no necesita tratar distinto un registro que ocupa
 mas de un bloque.

---
 emufs/tipo1.c | 54 +++++++++++++++++++++++----------------------------
 1 file changed, 24 insertions(+), 30 deletions(-)

diff --git a/emufs/tipo1.c b/emufs/tipo1.c
index 827571d..a547479 100644
--- a/emufs/tipo1.c
+++ b/emufs/tipo1.c
@@ -114,8 +114,13 @@ void* emufs_tipo1_leer_registro(EMUFS* efs, EMUFS_REG_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 = (char*) malloc(*reg_size);
+			registro = chunk_ptr = (char*) malloc(*reg_size);
 			if (registro == NULL) {
 				/* TODO Manejo de errores */
 				free(block);
@@ -123,37 +128,26 @@ void* emufs_tipo1_leer_registro(EMUFS* efs, EMUFS_REG_ID reg_id,
 				*err = 2; /* EMUFS_ERROR_OUT_OF_MEMORY */
 				return NULL;
 			}
-			/* Si el registro ocupa más de un bloque */
-			if (*reg_size > block_space) {
-				/* TODO */
-				/* 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 = registro;
-
-				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;
+			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;
 				}
-			} else {
-				memcpy(registro, block + offset, *reg_size);
 			}
 			break;
 		}
-- 
2.43.0