[KJ] [PATCH] cris: kgdb: remove double_this()

[KJ] [PATCH] cris: kgdb: remove double_this()

[Alexey Dobriyan]

[-- Attachment #1: Type: text/plain, Size: 617 bytes --]

* Doesn't have C&C warning.
* Unused.

Signed-off-by: Alexey Dobriyan <adobriyan@gmail.com>
---

 arch/cris/arch-v10/kernel/kgdb.c |    6 ------
 1 file changed, 6 deletions(-)

--- ./arch/cris/arch-v10/kernel/kgdb.c
+++ ./arch/cris/arch-v10/kernel/kgdb.c
@@ -569,12 +569,6 @@ gdb_cris_strtol (const char *s, char **e
 	return x;
 }
 
-int
-double_this(int x)
-{
-        return 2 * x;
-}
-
 /********************************* Register image ****************************/
 /* Copy the content of a register image into another. The size n is
    the size of the register image. Due to struct assignment generation of


[-- Attachment #2: Type: text/plain, Size: 168 bytes --]

_______________________________________________
Kernel-janitors mailing list
Kernel-janitors@lists.osdl.org
https://lists.osdl.org/mailman/listinfo/kernel-janitors

Re: [KJ] [PATCH] cris: kgdb: remove double_this()

[Ricardo Nabinger Sanchez]

[-- Attachment #1: Type: text/plain, Size: 371 bytes --]

Quoting  Alexey Dobriyan <adobriyan@gmail.com>
Sent on  Thu, 6 Oct 2005 00:45:41 +0400

Hello,

> -int
> -double_this(int x)
> -{
> -        return 2 * x;
> -}

I'm wondering why this "2*x" wrapper was ever made.

-- 
Ricardo Nabinger Sanchez
GNU/Linux #140696 [http://counter.li.org]
Slackware Linux + FreeBSD

  Left to themselves, things tend to go from bad to worse.

[-- Attachment #2: Type: text/plain, Size: 168 bytes --]

_______________________________________________
Kernel-janitors mailing list
Kernel-janitors@lists.osdl.org
https://lists.osdl.org/mailman/listinfo/kernel-janitors

Re: [KJ] [PATCH] cris: kgdb: remove double_this()

[Felipe W Damasio]

    Hi,

    Or better yet, even if it is actually necessary, why wasn't it made 
inline.

    Or even a define.

    Or...yeah, why was it made in the first place? :-)

    Cheers,

Felipe Damasio

Ricardo Nabinger Sanchez wrote:

>Quoting  Alexey Dobriyan <adobriyan@gmail.com>
>Sent on  Thu, 6 Oct 2005 00:45:41 +0400
>
>Hello,
>
>  
>
>>-int
>>-double_this(int x)
>>-{
>>-        return 2 * x;
>>-}
>>    
>>
>
>I'm wondering why this "2*x" wrapper was ever made.
>
>  
>
>------------------------------------------------------------------------
>
>_______________________________________________
>Kernel-janitors mailing list
>Kernel-janitors@lists.osdl.org
>https://lists.osdl.org/mailman/listinfo/kernel-janitors
>  
>
_______________________________________________
Kernel-janitors mailing list
Kernel-janitors@lists.osdl.org
https://lists.osdl.org/mailman/listinfo/kernel-janitors