Then that toSerializable() override is most likely a copy/paste error from
JsonLayout.

On 28 April 2017 at 09:15, Mikael Ståldal <[email protected]> wrote:

> YamlLayout always prefix each event with a line with "---". JsonLayout is a
> bit different since you don't want a comma before the first event.
>
> On Fri, Apr 28, 2017 at 4:02 PM, Matt Sicker <[email protected]> wrote:
>
> > The YamlLayout should probably separate events by "--" or whatever the
> page
> > separator token is in yaml (not a widely used feature).
> >
> > On 28 April 2017 at 03:14, Mikael Ståldal <[email protected]>
> > wrote:
> >
> > > YamlLayout has this method:
> > >
> > > @Override
> > > public void toSerializable(final LogEvent event, final Writer writer)
> > > throws IOException {
> > >     if (complete && eventCount > 0) {
> > >         writer.append(", ");
> > >     }
> > >     super.toSerializable(event, writer);
> > > }
> > >
> > > As far as I can see, it shouldn't be there since you don't separate
> > entries
> > > with comma in YAML. There is a similar method in JsonLayout, where it
> > does
> > > make sense. I guess this is a copy-and-paste mistake when making
> > YamlLayout
> > > with JsonLayout as template?
> > >
> > > (XmlLayout does not have any similar method.)
> > >
> > >
> > > --
> > > [image: MagineTV]
> > >
> > > *Mikael Ståldal*
> > > Senior software developer
> > >
> > > *Magine TV*
> > > [email protected]
> > > Grev Turegatan 3  | 114 46 Stockholm, Sweden  |   www.magine.com
> > >
> > > Privileged and/or Confidential Information may be contained in this
> > > message. If you are not the addressee indicated in this message
> > > (or responsible for delivery of the message to such a person), you may
> > not
> > > copy or deliver this message to anyone. In such case,
> > > you should destroy this message and kindly notify the sender by reply
> > > email.
> > >
> >
> >
> >
> > --
> > Matt Sicker <[email protected]>
> >
>
>
>
> --
> [image: MagineTV]
>
> *Mikael Ståldal*
> Senior software developer
>
> *Magine TV*
> [email protected]
> Grev Turegatan 3  | 114 46 Stockholm, Sweden  |   www.magine.com
>
> Privileged and/or Confidential Information may be contained in this
> message. If you are not the addressee indicated in this message
> (or responsible for delivery of the message to such a person), you may not
> copy or deliver this message to anyone. In such case,
> you should destroy this message and kindly notify the sender by reply
> email.
>



-- 
Matt Sicker <[email protected]>

Reply via email to