Create account / Log in

Tracker ids 3463, 3465 and 3472

Discussion area for the development team.

Moderators: uckelman, Tim M

Tracker ids 3463, 3465 and 3472

Postby bobd » March 15th, 2011, 10:40 pm

I have committed fixes for the following to bobd-bugfixes3.1

Bug 3463 - Infinite recursion if $PieceName$ specified as Text Label
SVN revision 7658
I think I am the only one who has hit this one.

Bug 3465 - Empty DynamicProperty causes IllegalStateException on loading
SVN revision 7659
This was traced to a propblem with SequenceEncoder that I think is the culprit for many of the "IllegalStateException No state for Decoder", and the older "NoSuchElementException in SequenceEncoder.Decoder.nextToken", bugs.

Bug 3472 - Initial value of DynamicProperty is not getting evaluated
SVN revision 7660
This was something I wanted, no one else has asked for it.

Should I mark these as TRIAGED? (I only have the option of TRIAGED, ASSIGNED OR RESOLVED)

And re: bug 3465 - Do you want me to mark those I think are duplicates as such?
bobd
 
Posts: 20
Joined: March 6th, 2011, 11:09 pm

Re: [messages] [Developers] Tracker ids 3463, 3465 and 3472

Postby Tim M » March 15th, 2011, 11:45 pm

Re 3465 way back here



viewtopic.php?f=5

&t=3368&p=19577#p19575



Just checking as this proposed fix was also for a NoSuchElementException in
SequenceEncoder.Decoder.nextXXXXX()and hopefully you aren't doing the same
thing :-)







-----Original Message-----
From: messages-bounces@vassalengine.org
[mailto:messages-bounces@vassalengine.org] On Behalf Of bobd
Sent: Tuesday, March 15, 2011 5:40 PM
To: messages@vassalengine.org
Subject: [messages] [Developers] Tracker ids 3463, 3465 and 3472



I have committed fixes for the following to bobd-bugfixes3.1



Bug 3463 - Infinite recursion if $PieceName$ specified as Text Label

SVN revision 7658

I think I am the only one who has hit this one.



Bug 3465 - Empty DynamicProperty causes IllegalStateException on loading

SVN revision 7659

This was traced to a propblem with SequenceEncoder that I think is the

culprit for many of the "IllegalStateException No state for Decoder",

and the older "NoSuchElementException in

SequenceEncoder.Decoder.nextToken", bugs.



Bug 3472 - Initial value of DynamicProperty is not getting evaluated

SVN revision 7660

This was something I wanted, no one else has asked for it.



Should I mark these as TRIAGED? (I only have the option of TRIAGED,

ASSIGNED OR RESOLVED)



And re: bug 3465 - Do you want me to mark those I think are duplicates

as such?



_______________________________________________

Read this topic online here:

viewtopic.php?p=22214#p22214

_______________________________________________

messages mailing list

messages@vassalengine.org

http://www.vassalengine.org/mailman/listinfo/messages
Tim,
Vassal Uber Geek/Guru

Problems? post your OS, Physical Mem, version of Vassal and Java plus the Module in question.
No developer can help with out that info, thx!
User avatar
Tim M
 
Posts: 1812
Joined: December 8th, 2007, 12:22 pm
Location: Earth

Re: Tracker ids 3463, 3465 and 3472

Postby bobd » March 16th, 2011, 3:58 pm

No, 2490 was appearing in mergeState, 3465 and related are appearing in the loading of a module that had been saved with either no initial value for a DynamicProperty or no label in a Labeler.
bobd
 
Posts: 20
Joined: March 6th, 2011, 11:09 pm

Re: [messages] [Developers] Tracker ids 3463, 3465 and 3472

Postby uckelman » March 17th, 2011, 11:17 am

Thus spake bobd:
> I have committed fixes for the following to bobd-bugfixes3.1
>
> Bug 3463 - Infinite recursion if $PieceName$ specified as Text Label
> SVN revision 7658
> I think I am the only one who has hit this one.
>
> Bug 3465 - Empty DynamicProperty causes IllegalStateException on loading
> SVN revision 7659
> This was traced to a propblem with SequenceEncoder that I think is the
> culprit for many of the "IllegalStateException No state for Decoder",
> and the older "NoSuchElementException in
> SequenceEncoder.Decoder.nextToken", bugs.
>
> Bug 3472 - Initial value of DynamicProperty is not getting evaluated
> SVN revision 7660
> This was something I wanted, no one else has asked for it.

I'll try to review these soon. I'm finding that I'm being overtaken
by events right now...

> Should I mark these as TRIAGED? (I only have the option of TRIAGED,
> ASSIGNED OR RESOLVED)

TRIAGED indicates that someone has looked at the bug, verified that it
is a bug, and also not a dupe of an existing bug.

ASSIGNED indicates that someone has comitted to working on the bug.
(However, take this with a grain of salt---there are many bugs assigned
to me which I intend to work on, but I'm not working on presently.)
If you're working on a bug, you should also assign it to yourself. Right
now, everything is assigned to me by default.

RESOLVED means that the bug is fixed and code committed to the repo.

> And re: bug 3465 - Do you want me to mark those I think are duplicates
> as such?
>

Yes, please do.

--
J.
User avatar
uckelman
Site Admin
 
Posts: 8797
Joined: December 10th, 2007, 9:48 am
Location: Durham, England

Re: [messages] [Developers] Tracker ids 3463, 3465 and 3472

Postby uckelman » March 17th, 2011, 10:02 pm

Thus spake bobd:
> I have committed fixes for the following to bobd-bugfixes3.1

I started looking at these, but ran into a problem: You didn't say
which revisions on your branch contain these fixes, and you didn't
include any log messages with your revisions so that I could tell
what the changes were for.

Which revision goes with which bug?
--
J.
User avatar
uckelman
Site Admin
 
Posts: 8797
Joined: December 10th, 2007, 9:48 am
Location: Durham, England

Re: Tracker ids 3463, 3465 and 3472

Postby bobd » March 17th, 2011, 11:12 pm

Sorry about no commit message, the eclipse plugin didn't prompt me for a message and I didn't think about it until after I had committed all of the changes. I will have to research how to add commit messages before I do anymore work.

Bug 3463 - Infinite recursion if $PieceName$ specified as Text Label
SVN revision 7658

Bug 3465 - Empty DynamicProperty causes IllegalStateException on loading
SVN revision 7659

Bug 3472 - Initial value of DynamicProperty is not getting evaluated
SVN revision 7660

Hopefully that is the information you need.

Bob
bobd
 
Posts: 20
Joined: March 6th, 2011, 11:09 pm

Re: [messages] [Developers] Re: Tracker ids 3463, 3465 and 3

Postby uckelman » March 19th, 2011, 3:31 pm

Thus spake bobd:
> Sorry about no commit message, the eclipse plugin didn't prompt me for a
> message and I didn't think about it until after I had committed all of
> the changes. I will have to research how to add commit messages before
> I do anymore work.
>
> Bug 3463 - Infinite recursion if $PieceName$ specified as Text Label
> SVN revision 7658
>
> Bug 3465 - Empty DynamicProperty causes IllegalStateException on loading
> SVN revision 7659

There's a problem with this commit, namely that you've entirely replaced
SequenceEncoder.java instead of commiting just the changed lines. I'm
not sure how you did that, as it's not what I would expect to happen by
default.

--
J.
User avatar
uckelman
Site Admin
 
Posts: 8797
Joined: December 10th, 2007, 9:48 am
Location: Durham, England

Re: [messages] [Developers] Re: Tracker ids 3463, 3465 and 3

Postby uckelman » March 19th, 2011, 3:57 pm

Thus spake Joel Uckelman:
> Thus spake bobd:
> > Sorry about no commit message, the eclipse plugin didn't prompt me for a
> > message and I didn't think about it until after I had committed all of
> > the changes. I will have to research how to add commit messages before
> > I do anymore work.
> >
> > Bug 3463 - Infinite recursion if $PieceName$ specified as Text Label
> > SVN revision 7658
> >
> > Bug 3465 - Empty DynamicProperty causes IllegalStateException on loading
> > SVN revision 7659
>
> There's a problem with this commit, namely that you've entirely replaced
> SequenceEncoder.java instead of commiting just the changed lines. I'm
> not sure how you did that, as it's not what I would expect to happen by
> default.
>

I see what's wrong: You're saving files with DOS newlines instead of
UNIX ones. All of our source files use UNIX newlines; if you're using
DOS newlines, it screws up diffs: What I was seeing when I merged this
change was that every single line in the file was different---due to
to each '\n' having been converted to '\r\n'.

There's a setting in Eclipse which you can change to avoid this problem:

http://www.sics.se/node/2108

--
J.
User avatar
uckelman
Site Admin
 
Posts: 8797
Joined: December 10th, 2007, 9:48 am
Location: Durham, England

Re: [messages] [Developers] Re: Tracker ids 3463, 3465 and 3

Postby uckelman » March 19th, 2011, 4:05 pm

Thus spake bobd:
>
> Bug 3465 - Empty DynamicProperty causes IllegalStateException on loading
> SVN revision 7659

This looks ok to me. Have you checked around to see if you can find any
modules which were relying on the old behavior?

--
J.
User avatar
uckelman
Site Admin
 
Posts: 8797
Joined: December 10th, 2007, 9:48 am
Location: Durham, England

Re: [messages] [Developers] Re: Tracker ids 3463, 3465 and 3

Postby uckelman » March 19th, 2011, 4:12 pm

Thus spake bobd:
>
> Bug 3465 - Empty DynamicProperty causes IllegalStateException on loading
> SVN revision 7659

Merged to 3.1@7664.

--
J.
User avatar
uckelman
Site Admin
 
Posts: 8797
Joined: December 10th, 2007, 9:48 am
Location: Durham, England

Re: [messages] [Developers] Re: Tracker ids 3463, 3465 and 3

Postby uckelman » March 19th, 2011, 4:52 pm

Thus spake Joel Uckelman:
> Thus spake bobd:
> >
> > Bug 3465 - Empty DynamicProperty causes IllegalStateException on loading
> > SVN revision 7659
>
> This looks ok to me. Have you checked around to see if you can find any
> modules which were relying on the old behavior?
>

BTW, this old thread is related to this bug:

viewtopic.php?f=5&t=2555&p=15962&hilit=NoSuchElementException#p15962

--
J.
User avatar
uckelman
Site Admin
 
Posts: 8797
Joined: December 10th, 2007, 9:48 am
Location: Durham, England

Re: Tracker ids 3463, 3465 and 3472

Postby bobd » March 19th, 2011, 11:06 pm

Sorry about the newline thing, I will look at my other changes and make sure all are just LFs.

I missed one issue in Vassal, Embelishment.oldGetType(). Nothing in Vassal uses it but I suppose some custom module might so I should make a change to that as well. Sorry about that.

I haven't checked non-Vassal code for use of SequenceEncoder, which modules should I check?
bobd
 
Posts: 20
Joined: March 6th, 2011, 11:09 pm

Re: [messages] [Developers] Re: Tracker ids 3463, 3465 and 3

Postby uckelman » March 19th, 2011, 11:29 pm

Thus spake bobd:
> Sorry about the newline thing, I will look at my other changes and make
> sure all are just LFs.
>
> I missed one issue in Vassal, Embelishment.oldGetType(). Nothing in
> Vassal uses it but I suppose some custom module might so I should make a
> change to that as well. Sorry about that.

This is in reference to which of the three bugs?

I checked the log for Embellishment. Embellishment.Ed.oldGetType()
I deprecated in 4062, back in 2008. It might well have been an
oversight that I didn't also deprecate Embellishment.oldGetType() at the
same time.

Generally, we do want to keep deprecated code working until we remove
it, to give anything which depends on it time to stop relying on it.

> I haven't checked non-Vassal code for use of SequenceEncoder, which
> modules should I check?
>

Practically any module with custom Buildables would use it.

--
J.
User avatar
uckelman
Site Admin
 
Posts: 8797
Joined: December 10th, 2007, 9:48 am
Location: Durham, England

Re: Tracker ids 3463, 3465 and 3472

Postby bobd » March 20th, 2011, 12:26 am

Have fixed the newlines, the tests had Windows newlines so I have changed those in rev 7666.

Embellishment.oldGetType() is fixed in rev 7667

I don't wish to appear stupid (ha!) but how do I know which modules have custom buildables? Do you download all the modules on the Vassal site to keep an eye on what custom code there is in use?
bobd
 
Posts: 20
Joined: March 6th, 2011, 11:09 pm

Re: Tracker ids 3463, 3465 and 3472

Postby bobd » March 20th, 2011, 12:28 am

Oh, I didn't answer your question about oldGetType() - thats a bug 3465 (SequenceEncoder) issue as it called
new SequenceEncoder(null, ';')
bobd
 
Posts: 20
Joined: March 6th, 2011, 11:09 pm

Next

Return to Developers

Who is online

Users browsing this forum: No registered users and 2 guests