diff options
| author | mame <mame@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2008-07-30 17:22:16 +0000 |
|---|---|---|
| committer | mame <mame@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2008-07-30 17:22:16 +0000 |
| commit | 3ab3ec62c32a7791706ade4052e65286c9be2f99 (patch) | |
| tree | 2a8999fde577987a8607fb999306d91311e8452a /ext/syck | |
| parent | 9a21e314727275cdcd931471aa6dc08ecc7a1e80 (diff) | |
| download | ruby-3ab3ec62c32a7791706ade4052e65286c9be2f99.tar.gz ruby-3ab3ec62c32a7791706ade4052e65286c9be2f99.tar.xz ruby-3ab3ec62c32a7791706ade4052e65286c9be2f99.zip | |
* ext/syck/rubyext.c (rb_syck_compile): remove meaningless branch
that misleads into thinking that the variable sav may be
uninitialized.
git-svn-id: http://svn.ruby-lang.org/repos/ruby/trunk@18282 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
Diffstat (limited to 'ext/syck')
| -rw-r--r-- | ext/syck/rubyext.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/ext/syck/rubyext.c b/ext/syck/rubyext.c index bdf3c8c79..cb3ad260e 100644 --- a/ext/syck/rubyext.c +++ b/ext/syck/rubyext.c @@ -104,7 +104,7 @@ rb_syck_compile(VALUE self, VALUE port) char *ret; VALUE bc; bytestring_t *sav = NULL; - void *data; + void *data = NULL; SyckParser *parser = syck_new_parser(); taint = syck_parser_assign_io(parser, &port); @@ -113,7 +113,8 @@ rb_syck_compile(VALUE self, VALUE port) syck_parser_implicit_typing( parser, 0 ); syck_parser_taguri_expansion( parser, 0 ); oid = syck_parse( parser ); - if (syck_lookup_sym( parser, oid, &data )) sav = data; + ASSERT(syck_lookup_sym( parser, oid, &data )); + sav = data; ret = S_ALLOCA_N( char, strlen( sav->buffer ) + 3 ); ret[0] = '\0'; |