search

pinojs / quick-format-unescaped

Solves a problem with util.format
MIT License
17 stars 13 forks source link

fix: pino issue 1054 #36

Closed climba03003 closed 3 years ago

climba03003 commented 3 years ago

Fixes pinojs/pino#1054

When we face undefined and null.

The current behavior will break after the string concat of string before that placeholder, but it do not update the lastPos. So, those string will be double counted.

The fix is moving the break before the string concat. So, it will not double count those string. As the new placeholder always placed the last, this movement will not affect the other test.

climba03003 commented 3 years ago

@davidmarkclements @mcollina Can you help to publish a release for this fix?

mcollina commented 3 years ago

@climba03003 I do not have publish rights, only @davidmarkclements has.