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:
> On 30 Oct 2014, at 12:29, Francis Galiegue <[e-mail]> wrote:
>>
>>            if Value is null:
>>              if Name exists in Target:
>>                remove the Name/Value pair from Target
>>              else:
>>                Target[Name] = MergePatch(Target[Name], Value)
>
> 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
>

Wee! Glad I asked, then... I'd have made a fundamental implementation mistake.

> You can also look at
>
> https://gist.github.com/cabo/7888768#file-merge-patch-rb
>
> which is the same thing in a language that is not indentation-sensitive.
>

Thanks for the pointer!

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)