Bug 169 : Operation path encoding as required by RFC6901

[dylanbehetre]

To fix the issue 169 referenced at https://github.com/eclipse-ee4j/jsonp/issues/169, this commit change the method diffObject in JsonPatchImpl.java:224.

Before to add the key in operation patch, this code encode it as required by RFC6901 - JsonPointer.

Signed-off-by: dbehetre [email protected]

[dylanbehetre]

The travis build with openjdk11 failed on JSR 374 (JSON Processing) API, I wasn't update this part. Any suggestion ?

https://travis-ci.org/eclipse-ee4j/jsonp/jobs/524021578

Build successfully in local with openjdk 11.0.2 and 1.8.0_202. I don't understand the travis build failed on 11.0.2..

[leadpony]

Hello @dylanbehetre
Please see #163

[m0mus]

It looks good to me, but I want to hear @lukasj opinion before merging it.

[m0mus]

Please update copyright years in all files you changed.

[dylanbehetre]

Please update copyright years in all files you changed.

I change the date on the right to 2019, is it ok for you ?

[m0mus]

@dylanbehetre yes

[keilw]

@m0mus Is @lukasj on vacation because he has not provided any input here?

[lukasj]

@keilw it’s in the queue. Or is this the only issue blocking Jakarta EE 8 work? Are we going to restage the release if this gets merged immediately?

[dylanbehetre]

I already did changes asking in requested changes on 22 July. I don't understand why the requested changes are always here.

[keilw]

What's the status here? Jakarta EE 8 is done, so it won't be until another update to Jakarta JSONP

[srius]

Hi every one,
Would it be possible to have an update on this PR state ? Change requested seems to be taken into account by Dylan...
Should the PR be adapted to comply to new package name ? What is expected from you ?

[m0mus]

LGTM