Re: [Json] RFC 7386: correct indentation of the MergePatch function...

[Francis Galiegue <[e-mail]>]

On Thu, Oct 30, 2014 at 12:37 PM, Carsten Bormann <[e-mail]> wrote:
[...]
>
> Unfortunately, not quite.
>
> http://tools.ietf.org/html/draft-ietf-appsawg-json-merge-patch-07
>
> says:
>
>    define MergePatch(Target, Patch):
>      if Patch is an Object:
>        if Target is not an Object:
>          Target = {} # Ignore the contents and set it to an empty Object
>        for each Name/Value pair in Patch:
>          if Value is null:
>            if Name exists in Target:
>              remove the Name/Value pair from Target
>          else:
>            Target[Name] = MergePatch(Target[Name], Value)
>        return Target
>      else:
>        return Patch
>

OK then, might I suggest that the pseudo-language used be independent
of the indentation? This could be another erratum, I guess, only I
don't even know how to "post" an erratum to begin with.

Regards,
-- 
Francis Galiegue, [e-mail], https://github.com/fge
JSON Schema in Java: http://json-schema-validator.herokuapp.com
Parsers in pure Java: https://github.com/parboiled1/grappa (redde
Caesaris: https://github.com/sirthias)