GNU bug report logs - #19172
[PATCH 1/1] Remove unused funcs that cause a building error

Previous Next

Package: guile;

Reported by: Pedro Aguilar <paguilar <at> paguilar.org>

Date: Tue, 25 Nov 2014 00:24:01 UTC

Severity: normal

Tags: patch

Done: Andy Wingo <wingo <at> pobox.com>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Pedro Aguilar <paguilar <at> paguilar.org>
Subject: bug#19172: closed (Re: bug#19172: [PATCH 1/1] Remove unused funcs
 that cause a building error)
Date: Wed, 22 Jun 2016 14:50:02 +0000
[Message part 1 (text/plain, inline)]
Your bug report

#19172: [PATCH 1/1] Remove unused funcs that cause a building error

which was filed against the guile package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to 19172 <at> debbugs.gnu.org.

-- 
19172: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=19172
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems
[Message part 2 (message/rfc822, inline)]
From: Andy Wingo <wingo <at> pobox.com>
To: Pedro Aguilar <paguilar <at> paguilar.org>
Cc: thomas.petazzoni <at> free-electrons.com, 19172-done <at> debbugs.gnu.org
Subject: Re: bug#19172: [PATCH 1/1] Remove unused funcs that cause a building
 error
Date: Wed, 22 Jun 2016 16:49:01 +0200
Thanks for the patch!  I applied a version that also deleted the
previous two static inline helpers.  Weird to have them around.  Will be
in 2.0.12 and 2.1.4.

Cheers,

Andy

On Tue 25 Nov 2014 00:14, Pedro Aguilar <paguilar <at> paguilar.org> writes:

> Remove unused static inline functions str_upcase_l() and
> str_downcase_l() that cause the compilation error
> 'dereferencing pointer to incomplete type' when building
> for ARM using a buildroot-generated toolchain.
>
> Signed-off-by: Pedro Aguilar <paguilar <at> paguilar.org>
> ---
>  libguile/i18n.c | 20 --------------------
>  1 file changed, 20 deletions(-)
>
> diff --git a/libguile/i18n.c b/libguile/i18n.c
> index c6b9b84..32278bd 100644
> --- a/libguile/i18n.c
> +++ b/libguile/i18n.c
> @@ -851,26 +851,6 @@ str_downcase (register char *dst, register const char *src)
>    *dst = '\0';
>  }
>  
> -#ifdef USE_GNU_LOCALE_API
> -static inline void
> -str_upcase_l (register char *dst, register const char *src,
> -	      scm_t_locale locale)
> -{
> -  for (; *src != '\0'; src++, dst++)
> -    *dst = toupper_l (*src, locale);
> -  *dst = '\0';
> -}
> -
> -static inline void
> -str_downcase_l (register char *dst, register const char *src,
> -		scm_t_locale locale)
> -{
> -  for (; *src != '\0'; src++, dst++)
> -    *dst = tolower_l (*src, locale);
> -  *dst = '\0';
> -}
> -#endif
> -
>  
>  SCM_DEFINE (scm_string_locale_lt, "string-locale<?", 2, 1, 0,
>  	    (SCM s1, SCM s2, SCM locale),

[Message part 3 (message/rfc822, inline)]
From: Pedro Aguilar <paguilar <at> paguilar.org>
To: bug-guile <at> gnu.org
Cc: thomas.petazzoni <at> free-electrons.com, Pedro Aguilar <paguilar <at> paguilar.org>
Subject: [PATCH 1/1] Remove unused funcs that cause a building error
Date: Tue, 25 Nov 2014 00:14:32 +0100
Remove unused static inline functions str_upcase_l() and
str_downcase_l() that cause the compilation error
'dereferencing pointer to incomplete type' when building
for ARM using a buildroot-generated toolchain.

Signed-off-by: Pedro Aguilar <paguilar <at> paguilar.org>
---
 libguile/i18n.c | 20 --------------------
 1 file changed, 20 deletions(-)

diff --git a/libguile/i18n.c b/libguile/i18n.c
index c6b9b84..32278bd 100644
--- a/libguile/i18n.c
+++ b/libguile/i18n.c
@@ -851,26 +851,6 @@ str_downcase (register char *dst, register const char *src)
   *dst = '\0';
 }
 
-#ifdef USE_GNU_LOCALE_API
-static inline void
-str_upcase_l (register char *dst, register const char *src,
-	      scm_t_locale locale)
-{
-  for (; *src != '\0'; src++, dst++)
-    *dst = toupper_l (*src, locale);
-  *dst = '\0';
-}
-
-static inline void
-str_downcase_l (register char *dst, register const char *src,
-		scm_t_locale locale)
-{
-  for (; *src != '\0'; src++, dst++)
-    *dst = tolower_l (*src, locale);
-  *dst = '\0';
-}
-#endif
-
 
 SCM_DEFINE (scm_string_locale_lt, "string-locale<?", 2, 1, 0,
 	    (SCM s1, SCM s2, SCM locale),
-- 
1.9.1




This bug report was last modified 9 years and 52 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.