Cannot deserialize type with generics about json-io

[jdereg]

This issue has been corrected and a new json-io 2.7.4 has been released to correct it. It should be available from Maven Repos after 13:00 Eastern Standard Time (US).

[seeingmachines-fovio]

Hi,

when I raised this issue, it was indeed about inner classes, but the issue still seems to remain for normal classes... Just make ADouble and UseADouble normal, public, non-inner classes

Cheers,
L.

[jdereg]

What is inside ADouble? Can you send me that?

On Mon, Dec 22, 2014 at 9:53 AM, seeingmachines-fovio <
[email protected]> wrote:

Hi,

when I raised this issue, it was indeed about inner classes, but the issue
still seems to remain for normal classes... Just make ADouble and
UseADouble normal, public, non-inner classes

Cheers,
L.

—
Reply to this email directly or view it on GitHub
#25 (comment).

[ludekdolejsky]

Hi,

I have updated the issue many times - you probably only saw my first edit?

Have a look here: #25 #25

public class ADouble {
private T field1;
private T field2;

public ADouble(T field1, T field2) {
    this.field1 = field1;
    this.field2 = field2;
}

}

public class UseADouble {
private ADouble a;

public UseADouble(ADouble<String> a) {
    this.a = a;
}

}

Cheers,
L.

On 22 Dec 2014, at 16:06, John DeRegnaucourt [email protected] wrote:

What is inside ADouble? Can you send me that?

On Mon, Dec 22, 2014 at 9:53 AM, seeingmachines-fovio <
[email protected]> wrote:

Hi,

when I raised this issue, it was indeed about inner classes, but the issue
still seems to remain for normal classes... Just make ADouble and
UseADouble normal, public, non-inner classes

Cheers,
L.

—
Reply to this email directly or view it on GitHub
#25 (comment).

—
Reply to this email directly or view it on GitHub #25 (comment).

[jdereg]

How did you test this, did you get the source code for JsonReader.java and
directly use it? Maven Repo's don't yet have 2.7.4 (they will in about 2
more hours).

On Mon, Dec 22, 2014 at 10:06 AM, John DeRegnaucourt [email protected]
wrote:

What is inside ADouble? Can you send me that?

On Mon, Dec 22, 2014 at 9:53 AM, seeingmachines-fovio <
[email protected]> wrote:

Hi,

when I raised this issue, it was indeed about inner classes, but the
issue still seems to remain for normal classes... Just make ADouble and
UseADouble normal, public, non-inner classes

Cheers,
L.

—
Reply to this email directly or view it on GitHub
#25 (comment).

[ludekdolejsky]

Yes, I just cloned fresh from master and ran the test

L.

On 22 Dec 2014, at 16:11, John DeRegnaucourt [email protected] wrote:

How did you test this, did you get the source code for JsonReader.java and
directly use it? Maven Repo's don't yet have 2.7.4 (they will in about 2
more hours).

On Mon, Dec 22, 2014 at 10:06 AM, John DeRegnaucourt [email protected]
wrote:

What is inside ADouble? Can you send me that?

On Mon, Dec 22, 2014 at 9:53 AM, seeingmachines-fovio <
[email protected]> wrote:

Hi,

when I raised this issue, it was indeed about inner classes, but the
issue still seems to remain for normal classes... Just make ADouble and
UseADouble normal, public, non-inner classes

Cheers,
L.

—
Reply to this email directly or view it on GitHub
#25 (comment).

—
Reply to this email directly or view it on GitHub #25 (comment).

[jdereg]

Ludek, thank you for the additional testing. I see the issue now, and plan
to create a 2.7.5 today. Adding additional tests.

On Mon, Dec 22, 2014 at 10:19 AM, Ludek Dolejsky [email protected]
wrote:

Yes, I just cloned fresh from master and ran the test

L.

On 22 Dec 2014, at 16:11, John DeRegnaucourt [email protected]
wrote:

How did you test this, did you get the source code for JsonReader.java
and
directly use it? Maven Repo's don't yet have 2.7.4 (they will in about 2
more hours).

On Mon, Dec 22, 2014 at 10:06 AM, John DeRegnaucourt [email protected]
wrote:

What is inside ADouble? Can you send me that?

On Mon, Dec 22, 2014 at 9:53 AM, seeingmachines-fovio <
[email protected]> wrote:

Hi,

when I raised this issue, it was indeed about inner classes, but the
issue still seems to remain for normal classes... Just make ADouble
and
UseADouble normal, public, non-inner classes

Cheers,
L.

—
Reply to this email directly or view it on GitHub
#25 (comment).

—
Reply to this email directly or view it on GitHub <
https://github.com/jdereg/json-io/issues/25#issuecomment-67847331>.

—
Reply to this email directly or view it on GitHub
#25 (comment).

[jdereg]

Ludek, I have created a 2.7.5 version of json-io which fixes the
ArrayIndexOutOfBoundsException. I added test with a multi-template class
that use two fields of one template type, and one of the other. The test
now passes. The ArrayIndexOutOfBounds exception will no longer occur,
however, I still have some work to do to perfect handling the associated
template variable to the associated value. What I have works pretty good,
and works in many cases you would not expect it to, because the type can be
figured out later. Let me know if you have any further issues related to
template support.

On Mon, Dec 22, 2014 at 10:21 AM, John DeRegnaucourt [email protected]
wrote:

Ludek, thank you for the additional testing. I see the issue now, and
plan to create a 2.7.5 today. Adding additional tests.

On Mon, Dec 22, 2014 at 10:19 AM, Ludek Dolejsky <[email protected]

wrote:

Yes, I just cloned fresh from master and ran the test

L.

On 22 Dec 2014, at 16:11, John DeRegnaucourt [email protected]
wrote:

How did you test this, did you get the source code for JsonReader.java
and
directly use it? Maven Repo's don't yet have 2.7.4 (they will in about
2
more hours).

On Mon, Dec 22, 2014 at 10:06 AM, John DeRegnaucourt [email protected]

wrote:

What is inside ADouble? Can you send me that?

On Mon, Dec 22, 2014 at 9:53 AM, seeingmachines-fovio <
[email protected]> wrote:

Hi,

when I raised this issue, it was indeed about inner classes, but the
issue still seems to remain for normal classes... Just make ADouble
and
UseADouble normal, public, non-inner classes

Cheers,
L.

—
Reply to this email directly or view it on GitHub
#25 (comment).

—
Reply to this email directly or view it on GitHub <
https://github.com/jdereg/json-io/issues/25#issuecomment-67847331>.

—
Reply to this email directly or view it on GitHub
#25 (comment).