On Fri, Feb 4, 2011 at 9:48 PM, Pang Yan Han <[email protected]> wrote:
> When reading the "desc" file in _alpm_local_db_read(), some
> strings are trimmed and checked for length > 0 before their
> use/duplication subsequently. They are then trimmed again
> when there is no need to.
>
> The following code snippet should illustrate it clearly:
>
> while(fgets(line, sizeof(line), fp) &&
> strlen(_alpm_strtrim(line))) {
>
> char *linedup;
> STRDUP(linedup, _alpm_strtrim(line), goto error);
> info->groups = alpm_list_add(info->groups, linedup);
> }
>
> This patch removes the redundant _alpm_strtrim() calls in
> _alpm_local_db_read() such as the one inside the STRDUP shown
> above.
>
> Signed-off-by: Pang Yan Han <[email protected]>
Thanks! Applied.
> ---
> lib/libalpm/be_local.c | 18 +++++++++---------
> 1 files changed, 9 insertions(+), 9 deletions(-)
>
> diff --git a/lib/libalpm/be_local.c b/lib/libalpm/be_local.c
> index c7110fa..c005532 100644
> --- a/lib/libalpm/be_local.c
> +++ b/lib/libalpm/be_local.c
> @@ -539,7 +539,7 @@ int _alpm_local_db_read(pmdb_t *db, pmpkg_t *info,
> pmdbinfrq_t inforeq)
> } else if(strcmp(line, "%GROUPS%") == 0) {
> while(fgets(line, sizeof(line), fp) &&
> strlen(_alpm_strtrim(line))) {
> char *linedup;
> - STRDUP(linedup, _alpm_strtrim(line),
> goto error);
> + STRDUP(linedup, line, goto error);
> info->groups =
> alpm_list_add(info->groups, linedup);
> }
> } else if(strcmp(line, "%URL%") == 0) {
> @@ -550,7 +550,7 @@ int _alpm_local_db_read(pmdb_t *db, pmpkg_t *info,
> pmdbinfrq_t inforeq)
> } else if(strcmp(line, "%LICENSE%") == 0) {
> while(fgets(line, sizeof(line), fp) &&
> strlen(_alpm_strtrim(line))) {
> char *linedup;
> - STRDUP(linedup, _alpm_strtrim(line),
> goto error);
> + STRDUP(linedup, line, goto error);
> info->licenses =
> alpm_list_add(info->licenses, linedup);
> }
> } else if(strcmp(line, "%ARCH%") == 0) {
> @@ -595,30 +595,30 @@ int _alpm_local_db_read(pmdb_t *db, pmpkg_t *info,
> pmdbinfrq_t inforeq)
> } else if(strcmp(line, "%REPLACES%") == 0) {
> while(fgets(line, sizeof(line), fp) &&
> strlen(_alpm_strtrim(line))) {
> char *linedup;
> - STRDUP(linedup, _alpm_strtrim(line),
> goto error);
> + STRDUP(linedup, line, goto error);
> info->replaces =
> alpm_list_add(info->replaces, linedup);
> }
> } else if(strcmp(line, "%DEPENDS%") == 0) {
> while(fgets(line, sizeof(line), fp) &&
> strlen(_alpm_strtrim(line))) {
> - pmdepend_t *dep =
> _alpm_splitdep(_alpm_strtrim(line));
> + pmdepend_t *dep =
> _alpm_splitdep(line);
> info->depends =
> alpm_list_add(info->depends, dep);
> }
> } else if(strcmp(line, "%OPTDEPENDS%") == 0) {
> while(fgets(line, sizeof(line), fp) &&
> strlen(_alpm_strtrim(line))) {
> char *linedup;
> - STRDUP(linedup, _alpm_strtrim(line),
> goto error);
> + STRDUP(linedup, line, goto error);
> info->optdepends =
> alpm_list_add(info->optdepends, linedup);
> }
> } else if(strcmp(line, "%CONFLICTS%") == 0) {
> while(fgets(line, sizeof(line), fp) &&
> strlen(_alpm_strtrim(line))) {
> char *linedup;
> - STRDUP(linedup, _alpm_strtrim(line),
> goto error);
> + STRDUP(linedup, line, goto error);
> info->conflicts =
> alpm_list_add(info->conflicts, linedup);
> }
> } else if(strcmp(line, "%PROVIDES%") == 0) {
> while(fgets(line, sizeof(line), fp) &&
> strlen(_alpm_strtrim(line))) {
> char *linedup;
> - STRDUP(linedup, _alpm_strtrim(line),
> goto error);
> + STRDUP(linedup, line, goto error);
> info->provides =
> alpm_list_add(info->provides, linedup);
> }
> }
> @@ -639,13 +639,13 @@ int _alpm_local_db_read(pmdb_t *db, pmpkg_t *info,
> pmdbinfrq_t inforeq)
> if(strcmp(line, "%FILES%") == 0) {
> while(fgets(line, sizeof(line), fp) &&
> strlen(_alpm_strtrim(line))) {
> char *linedup;
> - STRDUP(linedup, _alpm_strtrim(line),
> goto error);
> + STRDUP(linedup, line, goto error);
> info->files =
> alpm_list_add(info->files, linedup);
> }
> } else if(strcmp(line, "%BACKUP%") == 0) {
> while(fgets(line, sizeof(line), fp) &&
> strlen(_alpm_strtrim(line))) {
> char *linedup;
> - STRDUP(linedup, _alpm_strtrim(line),
> goto error);
> + STRDUP(linedup, line, goto error);
> info->backup =
> alpm_list_add(info->backup, linedup);
> }
> }
> --
> 1.7.4
>
>
>
