Re: Sets in libyaml output -- open PR on Bitbucket
Ingy dot Net <[email protected]>
| Newsgroups | gmane.text.yaml.general |
|---|---|
| Message-ID | <CAHJtQJ7mT4V_LkM+eaV1FFgvTuvyReC2npJMzaK+FfRpU4wdRg@mail.gmail.com> |
This patch is inappropriate for libyaml. A set is not a *kind* (mapping, sequence, scalar). Events are only for kinds. See: http://yaml.org/spec/1.2/spec.html#id2762357 Feel free to drop by irc.freenode.net#yaml if you wish to chat about it. On Sun, Aug 3, 2014 at 8:02 PM, Zenaan Harkness <[email protected]> wrote: > On 5/4/14, Susu Laird <[email protected]> wrote: > > Ping! Is anyone able to look at this? > > I don't know that this is the list specifically for libyaml, > not sure sorry. Other than that I'm not able to usefully > comment. > > If your "enhancement" matches the spec, then you are > actually fixing a bug, I would think, in the code you have > worked on. > > If you are actually enhancing, and not just fixing a bug, > then it seems you would be changing the spec. > > Since I don't know what the case is specifically, are you > able to comment on my comments? > > Good luck > Zenaan > > > ------------------------------------------------------------------------------ > Infragistics Professional > Build stunning WinForms apps today! > Reboot your WinForms applications with our WinForms controls. > Build a bridge from your legacy apps to the future. > > http://pubads.g.doubleclick.net/gampad/clk?id=153845071&iu=/4140/ostg.clktrk > _______________________________________________ > Yaml-core mailing list > [email protected] > https://lists.sourceforge.net/lists/listinfo/yaml-core > ------------------------------------------------------------------------------ Infragistics Professional Build stunning WinForms apps today! Reboot your WinForms applications with our WinForms controls. Build a bridge from your legacy apps to the future. http://pubads.g.doubleclick.net/gampad/clk?id=153845071&iu=/4140/ostg.clktrk _______________________________________________ Yaml-core mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/yaml-core