Comments (8)
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).
from json-io.
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.
from json-io.
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 classesCheers,
L.—
Reply to this email directly or view it on GitHub
#25 (comment).
from json-io.
Hi,
I have updated the issue many times - you probably only saw my first edit?
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 classesCheers,
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).
from json-io.
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 classesCheers,
L.—
Reply to this email directly or view it on GitHub
#25 (comment).
from json-io.
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 classesCheers,
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).
from json-io.
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 classesCheers,
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).
from json-io.
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 classesCheers,
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).
from json-io.
Related Issues (20)
- vulnerable to Denial Of Services (DoS) HOT 2
- Exception while deserializing `SingletonList` HOT 1
- Incorrect looking tag "14.4.2" for release version 4.14.2 HOT 1
- Possible issue about list of objects that have custom writers HOT 5
- Java class version 55 for 4.14.2 HOT 3
- Revisiting issue 17 HOT 6
- Enums are serialized differently with 4.15.0 HOT 3
- 4.15 --> 4.16 default mapping method fails to parse HOT 7
- Enhance the robustness of tests
- ReadOptionsBuilder HOT 1
- Compile / runtime dependency on com.novell.ldap:jldap HOT 3
- How to deserialise javafx classes with inaccessible fields? HOT 2
- Request for tutorial HOT 2
- NoSuchMethodError after upgrade HOT 24
- Custom serialising of objects with inheritance HOT 2
- Encoding Bug for JsonIo.toJson("🙊🙉🙈", null)? HOT 3
- Custom Writers not including id and ref in output? HOT 3
- Compatibility between 4.14.0 and 4.20.0 HOT 4
- User guide Untyped Usage HOT 1
- high usage of metaspace after upgrade to 4.22.0 HOT 4
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from json-io.