R devel - Aug 2010 - Handle RAWSXP in inspect.c:typename()

Hi all

I had written a gdb macro to dump the string representation of an SEXPREC
type when I realised everything I needed was in inspect.c already in the
typename() function. However, the typename function doesnt handle the RAWSXP
type, so if possible, could the following patch be applied (I've just put in
inline as it is a trivial 1-liner)?

Index: src/main/inspect.c
==================================================================---
src/main/inspect.c    (revision 52221)
+++ src/main/inspect.c    (working copy)
@@ -63,6 +63,7 @@
     case EXTPTRSXP:    return "EXTPTRSXP";
     case WEAKREFSXP:    return "WEAKREFSXP";
     case S4SXP:        return "S4SXP";
+    case RAWSXP:    return "RAWSXP";
     default:
     return "<unknown>";
     }


Best
-- Rory

	[[alternative HTML version deleted]]

On Aug 22, 2010, at 4:47 AM, Rory Winston wrote:
> Hi all
> 
> I had written a gdb macro to dump the string representation of an SEXPREC
> type when I realised everything I needed was in inspect.c already in the
> typename() function. However, the typename function doesnt handle the
RAWSXP
> type, so if possible, could the following patch be applied (I've just
put in
> inline as it is a trivial 1-liner)?
> 
... it' just a part of the story - RAWSXPs are not handled at all - not even
in inspect itself (I'm not sure how I missed that ;)). I have added RAW
support to inspect in R-devel now.

Thanks,
Simon

> Index: src/main/inspect.c
> ==================================================================> ---
src/main/inspect.c    (revision 52221)
> +++ src/main/inspect.c    (working copy)
> @@ -63,6 +63,7 @@
>     case EXTPTRSXP:    return "EXTPTRSXP";
>     case WEAKREFSXP:    return "WEAKREFSXP";
>     case S4SXP:        return "S4SXP";
> +    case RAWSXP:    return "RAWSXP";
>     default:
>     return "<unknown>";
>     }
> 
> 
> Best
> -- Rory
> 
> 	[[alternative HTML version deleted]]
> 
> ______________________________________________
> R-devel at r-project.org mailing list
> https://stat.ethz.ch/mailman/listinfo/r-devel
> 
>