Skip to content

Commit

Permalink
* ext/syck: suppress warnings.
Browse files Browse the repository at this point in the history 
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@18170 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
  • Loading branch information
@nobu
nobu committed Jul 22, 2008
1 parent a0dfa48 commit 5c0dcf5
Show file tree
Hide file tree
Showing 8 changed files with 76 additions and 57 deletions.
4 changes: 4 additions & 0 deletions ChangeLog
View file
@@ -1,3 +1,7 @@
Wed Jul 23 03:02:08 2008 Nobuyoshi Nakada <nobu@ruby-lang.org>

* ext/syck: suppress warnings.

Wed Jul 23 00:34:20 2008 Nobuyoshi Nakada <nobu@ruby-lang.org>

* ext/openssl: suppress warnings.
Expand Down
54 changes: 30 additions & 24 deletions ext/syck/emitter.c
View file
Expand Up @@ -538,7 +538,7 @@ void syck_emit_indent( SyckEmitter *e )
* Basic printable test for LATIN-1 characters.
*/
int
syck_scan_scalar( int req_width, char *cursor, long len )
syck_scan_scalar( int req_width, const char *cursor, long len )
{
long i = 0, start = 0;
int flags = SCAN_NONE;
Expand Down Expand Up @@ -638,13 +638,14 @@ syck_scan_scalar( int req_width, char *cursor, long len )
* All scalars should be emitted through this function, which determines an appropriate style,
* tag and indent.
*/
void syck_emit_scalar( SyckEmitter *e, char *tag, enum scalar_style force_style, int force_indent, int force_width,
char keep_nl, char *str, long len )
void syck_emit_scalar( SyckEmitter *e, const char *tag, enum scalar_style force_style, int force_indent, int force_width,
char keep_nl, const char *str, long len )
{
enum scalar_style favor_style = scalar_literal;
SyckLevel *parent = syck_emitter_parent_level( e );
SyckLevel *lvl = syck_emitter_current_level( e );
int scan = 0;
const char *match_implicit;
char *implicit;

if ( str == NULL ) str = "";
Expand All @@ -658,10 +659,10 @@ void syck_emit_scalar( SyckEmitter *e, char *tag, enum scalar_style force_style,
}

scan = syck_scan_scalar( force_width, str, len );
implicit = syck_match_implicit( str, len );
match_implicit = syck_match_implicit( str, len );

/* quote strings which default to implicits */
implicit = syck_taguri( YAML_DOMAIN, implicit, strlen( implicit ) );
implicit = syck_taguri( YAML_DOMAIN, match_implicit, strlen( match_implicit ) );
if ( syck_tagcmp( tag, implicit ) != 0 && syck_tagcmp( tag, "tag:yaml.org,2002:str" ) == 0 ) {
force_style = scalar_2quote;
} else {
Expand Down Expand Up @@ -772,7 +773,7 @@ void syck_emit_scalar( SyckEmitter *e, char *tag, enum scalar_style force_style,
}

void
syck_emitter_escape( SyckEmitter *e, char *src, long len )
syck_emitter_escape( SyckEmitter *e, const char *src, long len )
{
int i;
for( i = 0; i < len; i++ )
Expand Down Expand Up @@ -801,12 +802,13 @@ syck_emitter_escape( SyckEmitter *e, char *src, long len )
/*
* Outputs a single-quoted block.
*/
void syck_emit_1quoted( SyckEmitter *e, int width, char *str, long len )
void
syck_emit_1quoted( SyckEmitter *e, int width, const char *str, long len )
{
char do_indent = 0;
char *mark = str;
char *start = str;
char *end = str;
const char *mark = str;
const char *start = str;
const char *end = str;
syck_emitter_write( e, "'", 1 );
while ( mark < str + len ) {
if ( do_indent ) {
Expand Down Expand Up @@ -848,12 +850,13 @@ void syck_emit_1quoted( SyckEmitter *e, int width, char *str, long len )
/*
* Outputs a double-quoted block.
*/
void syck_emit_2quoted( SyckEmitter *e, int width, char *str, long len )
void
syck_emit_2quoted( SyckEmitter *e, int width, const char *str, long len )
{
char do_indent = 0;
char *mark = str;
char *start = str;
char *end = str;
const char *mark = str;
const char *start = str;
const char *end = str;
syck_emitter_write( e, "\"", 1 );
while ( mark < str + len ) {
if ( do_indent > 0 ) {
Expand Down Expand Up @@ -908,11 +911,12 @@ void syck_emit_2quoted( SyckEmitter *e, int width, char *str, long len )
/*
* Outputs a literal block.
*/
void syck_emit_literal( SyckEmitter *e, char keep_nl, char *str, long len )
void
syck_emit_literal( SyckEmitter *e, char keep_nl, const char *str, long len )
{
char *mark = str;
char *start = str;
char *end = str;
const char *mark = str;
const char *start = str;
const char *end = str;
syck_emitter_write( e, "|", 1 );
if ( keep_nl == NL_CHOMP ) {
syck_emitter_write( e, "-", 1 );
Expand Down Expand Up @@ -943,11 +947,12 @@ void syck_emit_literal( SyckEmitter *e, char keep_nl, char *str, long len )
/*
* Outputs a folded block.
*/
void syck_emit_folded( SyckEmitter *e, int width, char keep_nl, char *str, long len )
void
syck_emit_folded( SyckEmitter *e, int width, char keep_nl, const char *str, long len )
{
char *mark = str;
char *start = str;
char *end = str;
const char *mark = str;
const char *start = str;
const char *end = str;
syck_emitter_write( e, ">", 1 );
if ( keep_nl == NL_CHOMP ) {
syck_emitter_write( e, "-", 1 );
Expand Down Expand Up @@ -992,7 +997,7 @@ void syck_emit_folded( SyckEmitter *e, int width, char keep_nl, char *str, long
/*
* Begins emission of a sequence.
*/
void syck_emit_seq( SyckEmitter *e, char *tag, enum seq_style style )
void syck_emit_seq( SyckEmitter *e, const char *tag, enum seq_style style )
{
SyckLevel *parent = syck_emitter_parent_level( e );
SyckLevel *lvl = syck_emitter_current_level( e );
Expand All @@ -1013,7 +1018,8 @@ void syck_emit_seq( SyckEmitter *e, char *tag, enum seq_style style )
/*
* Begins emission of a mapping.
*/
void syck_emit_map( SyckEmitter *e, char *tag, enum map_style style )
void
syck_emit_map( SyckEmitter *e, const char *tag, enum map_style style )
{
SyckLevel *parent = syck_emitter_parent_level( e );
SyckLevel *lvl = syck_emitter_current_level( e );
Expand Down
7 changes: 4 additions & 3 deletions ext/syck/implicit.c
View file
Expand Up @@ -20,7 +20,7 @@
void
try_tag_implicit( SyckNode *n, int taguri )
{
char *tid = "";
const char *tid = "";
switch ( n->kind )
{
case syck_str_kind:
Expand All @@ -44,9 +44,10 @@ try_tag_implicit( SyckNode *n, int taguri )
}
}

char *syck_match_implicit( char *str, size_t len )
const char *
syck_match_implicit( const char *str, size_t len )
{
char *cursor, *limit, *marker;
const char *cursor, *limit, *marker;
cursor = str;
limit = str + len;

Expand Down
11 changes: 6 additions & 5 deletions ext/syck/rubyext.c
View file
Expand Up @@ -71,7 +71,7 @@ static VALUE syck_node_transform( VALUE );
* handler prototypes
*/
SYMID rb_syck_load_handler _((SyckParser *, SyckNode *));
void rb_syck_err_handler _((SyckParser *, char *));
void rb_syck_err_handler _((SyckParser *, const char *));
SyckNode * rb_syck_bad_anchor_handler _((SyckParser *, char *));
void rb_syck_output_handler _((SyckEmitter *, char *, long));
void rb_syck_emitter_handler _((SyckEmitter *, st_data_t));
Expand Down Expand Up @@ -103,7 +103,8 @@ rb_syck_compile(VALUE self, VALUE port)
int taint;
char *ret;
VALUE bc;
bytestring_t *sav;
bytestring_t *sav;
void *data;

SyckParser *parser = syck_new_parser();
taint = syck_parser_assign_io(parser, &port);
Expand All @@ -112,7 +113,7 @@ rb_syck_compile(VALUE self, VALUE port)
syck_parser_implicit_typing( parser, 0 );
syck_parser_taguri_expansion( parser, 0 );
oid = syck_parse( parser );
syck_lookup_sym( parser, oid, &sav );
if (syck_lookup_sym( parser, oid, &data )) sav = data;

ret = S_ALLOCA_N( char, strlen( sav->buffer ) + 3 );
ret[0] = '\0';
Expand Down Expand Up @@ -634,7 +635,7 @@ rb_syck_load_handler(SyckParser *p, SyckNode *n)
* friendly errors.
*/
void
rb_syck_err_handler(SyckParser *p, char *msg)
rb_syck_err_handler(SyckParser *p, const char *msg)
{
char *endl = p->cursor;

Expand Down Expand Up @@ -1189,7 +1190,7 @@ syck_resolver_tagurize(VALUE self, VALUE val)
VALUE
syck_defaultresolver_detect_implicit(VALUE self, VALUE val)
{
char *type_id;
const char *type_id;
VALUE tmp = rb_check_string_type(val);

if ( !NIL_P(tmp) )
Expand Down
12 changes: 8 additions & 4 deletions ext/syck/syck.c
View file
Expand Up @@ -177,7 +177,7 @@ syck_new_parser(void)
}

int
syck_add_sym( SyckParser *p, char *data )
syck_add_sym( SyckParser *p, void *data )
{
SYMID id = 0;
if ( p->syms == NULL )
Expand All @@ -190,10 +190,14 @@ syck_add_sym( SyckParser *p, char *data )
}

int
syck_lookup_sym( SyckParser *p, SYMID id, void *data )
syck_lookup_sym( SyckParser *p, SYMID id, void **datap )
{
st_data_t data = (st_data_t)*datap;
int ret;
if ( p->syms == NULL ) return 0;
return st_lookup( p->syms, id, data );
ret = st_lookup( p->syms, id, &data );
*datap = (void *)data;
return ret;
}

int
Expand Down Expand Up @@ -493,7 +497,7 @@ syck_parse( SyckParser *p )
}

void
syck_default_error_handler( SyckParser *p, char *msg )
syck_default_error_handler( SyckParser *p, const char *msg )
{
printf( "Error at [Line %d, Col %d]: %s\n",
p->linect,
Expand Down
30 changes: 15 additions & 15 deletions ext/syck/syck.h
View file
Expand Up @@ -155,7 +155,7 @@ typedef struct _syck_str SyckIoStr;
typedef struct _syck_level SyckLevel;

typedef SYMID (*SyckNodeHandler)(SyckParser *, SyckNode *);
typedef void (*SyckErrorHandler)(SyckParser *, char *);
typedef void (*SyckErrorHandler)(SyckParser *, const char *);
typedef SyckNode * (*SyckBadAnchorHandler)(SyckParser *, char *);
typedef long (*SyckIoFileRead)(char *, SyckIoFile *, long, long);
typedef long (*SyckIoStrRead)(char *, SyckIoStr *, long, long);
Expand Down Expand Up @@ -348,12 +348,12 @@ void syck_add_transfer( char *, SyckNode *, int );
char *syck_xprivate( char *, int );
char *syck_taguri( const char *, const char *, int );
int syck_tagcmp( const char *, const char * );
int syck_add_sym( SyckParser *, char * );
int syck_lookup_sym( SyckParser *, SYMID, void * );
int syck_add_sym( SyckParser *, void * );
int syck_lookup_sym( SyckParser *, SYMID, void ** );
int syck_try_implicit( SyckNode * );
char *syck_type_id_to_uri( char * );
void try_tag_implicit( SyckNode *, int );
char *syck_match_implicit( char *, size_t );
const char *syck_match_implicit( const char *, size_t );

/*
* API prototypes
Expand All @@ -371,17 +371,17 @@ void syck_emitter_handler( SyckEmitter *, SyckEmitterHandler );
void syck_free_emitter( SyckEmitter * );
void syck_emitter_clear( SyckEmitter * );
void syck_emitter_write( SyckEmitter *, const char *, long );
void syck_emitter_escape( SyckEmitter *, char *, long );
void syck_emitter_escape( SyckEmitter *, const char *, long );
void syck_emitter_flush( SyckEmitter *, long );
void syck_emit( SyckEmitter *, st_data_t );
void syck_emit_scalar( SyckEmitter *, char *, enum scalar_style, int, int, char, char *, long );
void syck_emit_1quoted( SyckEmitter *, int, char *, long );
void syck_emit_2quoted( SyckEmitter *, int, char *, long );
void syck_emit_folded( SyckEmitter *, int, char, char *, long );
void syck_emit_literal( SyckEmitter *, char, char *, long );
void syck_emit_seq( SyckEmitter *, char *, enum seq_style );
void syck_emit_scalar( SyckEmitter *, const char *, enum scalar_style, int, int, char, const char *, long );
void syck_emit_1quoted( SyckEmitter *, int, const char *, long );
void syck_emit_2quoted( SyckEmitter *, int, const char *, long );
void syck_emit_folded( SyckEmitter *, int, char, const char *, long );
void syck_emit_literal( SyckEmitter *, char, const char *, long );
void syck_emit_seq( SyckEmitter *, const char *, enum seq_style );
void syck_emit_item( SyckEmitter *, st_data_t );
void syck_emit_map( SyckEmitter *, char *, enum map_style );
void syck_emit_map( SyckEmitter *, const char *, enum map_style );
void syck_emit_end( SyckEmitter * );
void syck_emit_tag( SyckEmitter *, const char *, const char * );
void syck_emit_indent( SyckEmitter * );
Expand All @@ -395,7 +395,7 @@ void syck_free_parser( SyckParser * );
void syck_parser_set_root_on_error( SyckParser *, SYMID );
void syck_parser_implicit_typing( SyckParser *, int );
void syck_parser_taguri_expansion( SyckParser *, int );
int syck_scan_scalar( int, char *, long );
int syck_scan_scalar( int, const char *, long );
void syck_parser_handler( SyckParser *, SyckNodeHandler );
void syck_parser_error_handler( SyckParser *, SyckErrorHandler );
void syck_parser_bad_anchor_handler( SyckParser *, SyckBadAnchorHandler );
Expand All @@ -410,7 +410,7 @@ void free_any_io( SyckParser * );
long syck_parser_read( SyckParser * );
long syck_parser_readlen( SyckParser *, long );
SYMID syck_parse( SyckParser * );
void syck_default_error_handler( SyckParser *, char * );
void syck_default_error_handler( SyckParser *, const char * );
SYMID syck_yaml2byte_handler( SyckParser *, SyckNode * );
char *syck_yaml2byte( char * );

Expand Down Expand Up @@ -445,7 +445,7 @@ long syck_seq_count( SyckNode * );
/*
* Lexer prototypes
*/
void syckerror( char * );
void syckerror( const char * );
int syckparse( void * );
union YYSTYPE;
int sycklex( union YYSTYPE *, SyckParser * );
Expand Down
2 changes: 1 addition & 1 deletion ext/syck/token.c
View file
Expand Up @@ -2713,7 +2713,7 @@ syckwrap()
}

void
syckerror( char *msg )
syckerror( const char *msg )
{
if ( syck_parser_ptr->error_handler == NULL )
syck_parser_ptr->error_handler = syck_default_error_handler;
Expand Down
13 changes: 8 additions & 5 deletions ext/syck/yaml2byte.c
View file
Expand Up @@ -128,6 +128,7 @@ syck_yaml2byte_handler(p, n)
char *finish;
bytestring_t *val = NULL;
bytestring_t *sav = NULL;
void *data;
/*TRACE0("syck_yaml2byte_handler()");*/
val = bytestring_alloc();
if(n->anchor) bytestring_append(val,YAMLBYTE_ANCHOR, n->anchor, NULL);
Expand Down Expand Up @@ -188,7 +189,7 @@ syck_yaml2byte_handler(p, n)
for ( i = 0; i < n->data.list->idx; i++ )
{
oid = syck_seq_read( n, i );
syck_lookup_sym( p, oid, &sav );
if (syck_lookup_sym( p, oid, &data )) sav = data;
bytestring_extend(val, sav);
}
bytestring_append(val,YAMLBYTE_END_BRANCH,NULL,NULL);
Expand All @@ -198,10 +199,10 @@ syck_yaml2byte_handler(p, n)
for ( i = 0; i < n->data.pairs->idx; i++ )
{
oid = syck_map_read( n, map_key, i );
syck_lookup_sym( p, oid, &sav );
if (syck_lookup_sym( p, oid, &data )) sav = data;
bytestring_extend(val, sav);
oid = syck_map_read( n, map_value, i );
syck_lookup_sym( p, oid, &sav );
if (syck_lookup_sym( p, oid, &data )) sav = data;
bytestring_extend(val, sav);
}
bytestring_append(val,YAMLBYTE_END_BRANCH,NULL,NULL);
Expand All @@ -217,7 +218,8 @@ syck_yaml2byte(char *yamlstr)
{
SYMID oid;
char *ret;
bytestring_t *sav;
bytestring_t *sav;
void *data;

SyckParser *parser = syck_new_parser();
syck_parser_str_auto( parser, yamlstr, NULL );
Expand All @@ -227,7 +229,8 @@ syck_yaml2byte(char *yamlstr)
syck_parser_taguri_expansion( parser, 1 );
oid = syck_parse( parser );

if ( syck_lookup_sym( parser, oid, &sav ) == 1 ) {
if ( syck_lookup_sym( parser, oid, &data ) ) {
sav = data;
ret = S_ALLOC_N( char, strlen( sav->buffer ) + 3 );
ret[0] = '\0';
strcat( ret, "D\n" );
Expand Down

0 comments on commit 5c0dcf5

Please sign in to comment.