ruby-core@ruby-lang.org archive (unofficial mirror)
 help / color / mirror / Atom feed
* [ruby-core:74474] [Ruby trunk Bug#12203] dumper.c static int yaml_emitter_dump_scalar two variables have the same value
       [not found] <redmine.issue-12203.20160320122950@ruby-lang.org>
@ 2016-03-20 12:29 ` fanantoxa
  2016-03-22 11:49 ` [ruby-core:74488] [Ruby trunk Bug#12203][Assigned] " shyouhei
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 4+ messages in thread
From: fanantoxa @ 2016-03-20 12:29 UTC (permalink / raw)
  To: ruby-core

Issue #12203 has been reported by Anton Sivakov.

----------------------------------------
Bug #12203: dumper.c  static int yaml_emitter_dump_scalar two variables have the same value
https://bugs.ruby-lang.org/issues/12203

* Author: Anton Sivakov
* Status: Open
* Priority: Normal
* Assignee: 
* ruby -v: 
* Backport: 2.1: UNKNOWN, 2.2: UNKNOWN, 2.3: UNKNOWN
----------------------------------------
~~~
static int
yaml_emitter_dump_scalar(
.....
int plain_implicit = (strcmp((char *)node->tag,
               YAML_DEFAULT_SCALAR_TAG) == 0);
int quoted_implicit = (strcmp((char *)node->tag,
                YAML_DEFAULT_SCALAR_TAG) == 0);
~~~

variables 'plain_implicit' and 'quoted_implicit' have the same values. It possible wrong expression or this code can be optimized.



-- 
https://bugs.ruby-lang.org/

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [ruby-core:74488] [Ruby trunk Bug#12203][Assigned] dumper.c static int yaml_emitter_dump_scalar two variables have the same value
       [not found] <redmine.issue-12203.20160320122950@ruby-lang.org>
  2016-03-20 12:29 ` [ruby-core:74474] [Ruby trunk Bug#12203] dumper.c static int yaml_emitter_dump_scalar two variables have the same value fanantoxa
@ 2016-03-22 11:49 ` shyouhei
  2017-06-07 17:58 ` [ruby-core:81609] [Ruby trunk Bug#12203] " fanantoxa
  2017-06-08 14:03 ` [ruby-core:81621] [Ruby trunk Bug#12203][Third Party's Issue] " nobu
  3 siblings, 0 replies; 4+ messages in thread
From: shyouhei @ 2016-03-22 11:49 UTC (permalink / raw)
  To: ruby-core

Issue #12203 has been updated by Shyouhei Urabe.

Status changed from Open to Assigned
Assignee set to Aaron Patterson

They are passed to a macro.  I guess the author intentionally separated these two because merging them makes the code less intuitive.  It seems OK to be optimized.

----------------------------------------
Bug #12203: dumper.c  static int yaml_emitter_dump_scalar two variables have the same value
https://bugs.ruby-lang.org/issues/12203#change-57609

* Author: Anton Sivakov
* Status: Assigned
* Priority: Normal
* Assignee: Aaron Patterson
* ruby -v: 
* Backport: 2.1: UNKNOWN, 2.2: UNKNOWN, 2.3: UNKNOWN
----------------------------------------
~~~
static int
yaml_emitter_dump_scalar(
.....
int plain_implicit = (strcmp((char *)node->tag,
               YAML_DEFAULT_SCALAR_TAG) == 0);
int quoted_implicit = (strcmp((char *)node->tag,
                YAML_DEFAULT_SCALAR_TAG) == 0);
~~~

variables 'plain_implicit' and 'quoted_implicit' have the same values. It possible wrong expression or this code can be optimized.



-- 
https://bugs.ruby-lang.org/

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [ruby-core:81609] [Ruby trunk Bug#12203] dumper.c static int yaml_emitter_dump_scalar two variables have the same value
       [not found] <redmine.issue-12203.20160320122950@ruby-lang.org>
  2016-03-20 12:29 ` [ruby-core:74474] [Ruby trunk Bug#12203] dumper.c static int yaml_emitter_dump_scalar two variables have the same value fanantoxa
  2016-03-22 11:49 ` [ruby-core:74488] [Ruby trunk Bug#12203][Assigned] " shyouhei
@ 2017-06-07 17:58 ` fanantoxa
  2017-06-08 14:03 ` [ruby-core:81621] [Ruby trunk Bug#12203][Third Party's Issue] " nobu
  3 siblings, 0 replies; 4+ messages in thread
From: fanantoxa @ 2017-06-07 17:58 UTC (permalink / raw)
  To: ruby-core

Issue #12203 has been updated by fanantoxa (Anton Sivakov).


shyouhei (Shyouhei Urabe) wrote:
> They are passed to a macro.  I guess the author intentionally separated these two because merging them makes the code less intuitive.  It seems OK to be optimized.

Maybe reassign to someone else? Or at lease chen that this still here?

----------------------------------------
Bug #12203: dumper.c  static int yaml_emitter_dump_scalar two variables have the same value
https://bugs.ruby-lang.org/issues/12203#change-65307

* Author: fanantoxa (Anton Sivakov)
* Status: Assigned
* Priority: Normal
* Assignee: tenderlovemaking (Aaron Patterson)
* Target version: 
* ruby -v: 
* Backport: 2.1: UNKNOWN, 2.2: UNKNOWN, 2.3: UNKNOWN
----------------------------------------
~~~
static int
yaml_emitter_dump_scalar(
.....
int plain_implicit = (strcmp((char *)node->tag,
               YAML_DEFAULT_SCALAR_TAG) == 0);
int quoted_implicit = (strcmp((char *)node->tag,
                YAML_DEFAULT_SCALAR_TAG) == 0);
~~~

variables 'plain_implicit' and 'quoted_implicit' have the same values. It possible wrong expression or this code can be optimized.



-- 
https://bugs.ruby-lang.org/

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [ruby-core:81621] [Ruby trunk Bug#12203][Third Party's Issue] dumper.c static int yaml_emitter_dump_scalar two variables have the same value
       [not found] <redmine.issue-12203.20160320122950@ruby-lang.org>
                   ` (2 preceding siblings ...)
  2017-06-07 17:58 ` [ruby-core:81609] [Ruby trunk Bug#12203] " fanantoxa
@ 2017-06-08 14:03 ` nobu
  3 siblings, 0 replies; 4+ messages in thread
From: nobu @ 2017-06-08 14:03 UTC (permalink / raw)
  To: ruby-core

Issue #12203 has been updated by nobu (Nobuyoshi Nakada).

Status changed from Assigned to Third Party's Issue

Please report it to the upstream, https://github.com/ruby/psych.

----------------------------------------
Bug #12203: dumper.c  static int yaml_emitter_dump_scalar two variables have the same value
https://bugs.ruby-lang.org/issues/12203#change-65319

* Author: fanantoxa (Anton Sivakov)
* Status: Third Party's Issue
* Priority: Normal
* Assignee: tenderlovemaking (Aaron Patterson)
* Target version: 
* ruby -v: 
* Backport: 2.1: UNKNOWN, 2.2: UNKNOWN, 2.3: UNKNOWN
----------------------------------------
~~~
static int
yaml_emitter_dump_scalar(
.....
int plain_implicit = (strcmp((char *)node->tag,
               YAML_DEFAULT_SCALAR_TAG) == 0);
int quoted_implicit = (strcmp((char *)node->tag,
                YAML_DEFAULT_SCALAR_TAG) == 0);
~~~

variables 'plain_implicit' and 'quoted_implicit' have the same values. It possible wrong expression or this code can be optimized.



-- 
https://bugs.ruby-lang.org/

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2017-06-08 14:03 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <redmine.issue-12203.20160320122950@ruby-lang.org>
2016-03-20 12:29 ` [ruby-core:74474] [Ruby trunk Bug#12203] dumper.c static int yaml_emitter_dump_scalar two variables have the same value fanantoxa
2016-03-22 11:49 ` [ruby-core:74488] [Ruby trunk Bug#12203][Assigned] " shyouhei
2017-06-07 17:58 ` [ruby-core:81609] [Ruby trunk Bug#12203] " fanantoxa
2017-06-08 14:03 ` [ruby-core:81621] [Ruby trunk Bug#12203][Third Party's Issue] " nobu

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).