Juergen,

When did you try to fix it? We are not working on the /real/ head, here, but have created our own branch. Last update on our side was from last monday, 10:22am CET

Martijn


On 11/18/05, Juergen Donnerstag <[EMAIL PROTECTED]> wrote:
At least I tried to fixed it. There should be a proper exception and a
test for it now.

Though isFullRender can not replace markupStreamPos and though IMO it
introduces some redundancy (because though not completely true it
basically is markupStreamPos < 0), it offers additional flexibility
and the code is more readable. This change is not yet committed.

Juergen

On 11/18/05, Martijn Dashorst < [EMAIL PROTECTED]> wrote:
> The markupStreamPos was causing the panel to re-iterate the render process
> until the heapspace error occurred. The problem is, that when the second
> time the wicket:id="foo" is processed, the component points to earlier in
> the markupstream. Thus never ending the loop.
>
> Martijn
>
>
>
> On 11/18/05, Johan Compagner <[EMAIL PROTECTED]> wrote:
> > i thought juergen did fix this ?
> > There is a test now if it was rendered twice Or is that test to late?
> >
> > I still think that a isFullRender() from the request cycle instead of that
> markupstreamPos test is a better thing to do.
> >
> > johan
> >
> >
> >
> >
> > On 11/18/05, SourceForge.net < [EMAIL PROTECTED]> wrote:
> > > Bugs item #1359899, was opened at 2005-11-18 12:52
> > > Message generated for change (Tracker Item Submitted) made by Item
> Submitter
> > > You can respond by visiting:
> > >
> https://sourceforge.net/tracker/?func=detail&atid=684975&aid=1359899&group_id=119783
> > >
> > > Please note that this message will contain a full copy of the comment
> thread,
> > > including the initial issue submission, for this request,
> > > not just the latest update.
> > > Category: core
> > > Group: 1.2
> > > Status: Open
> > > Resolution: None
> > > Priority: 9
> > > Submitted By: Martijn Dashorst (dashorst)
> > > Assigned to: Nobody/Anonymous (nobody)
> > > Summary: Out of heapspace error while reusing component instance
> > >
> > > Initial Comment:
> > > This code example should give a wicket error, but
> > > instead gives a HTTP ERROR: 500 Java+heap+space.
> > >
> > > <html>
> > >     <body>
> > >         <span wicket:id="foo">foo</span>
> > >         <span wicket:id="foo">bar</span>
> > >     </body>
> > > </html>
> > >
> > >
> > > import wicket.markup.html.basic.Label ;
> > >
> > > public class Index extends QuickStartPage
> > > {
> > >     public Index()
> > >     {
> > >         add(new Label("foo", "die, Die, DIE!"));
> > >     }
> > > }
> > >
> > >
> > >
> ----------------------------------------------------------------------
> > >
> > > You can respond by visiting:
> > >
> https://sourceforge.net/tracker/?func=detail&atid=684975&aid=1359899&group_id=119783
> > >
> > >
> > > -------------------------------------------------------
> > > This SF.Net email is sponsored by the JBoss Inc.  Get Certified Today
> > > Register for a JBoss Training Course.  Free Certification Exam
> > > for All Training Attendees Through End of 2005. For more info visit:
> > > http://ads.osdn.com/?ad_id=7628&alloc_id=16845&op=click
> > > _______________________________________________
> > > Wicket-develop mailing list
> > > [email protected]
> > >
> https://lists.sourceforge.net/lists/listinfo/wicket-develop
> > >
> >
> >
>
>
>
> --
> Living a wicket life...
>
> Martijn Dashorst - http://www.jroller.com/page/dashorst
>
> Wicket 1.1 is out: http://wicket.sourceforge.net/wicket-1.1


-------------------------------------------------------
This SF.Net email is sponsored by the JBoss Inc.  Get Certified Today
Register for a JBoss Training Course.  Free Certification Exam
for All Training Attendees Through End of 2005. For more info visit:
http://ads.osdn.com/?ad_idv28&alloc_id845&opclick
_______________________________________________
Wicket-develop mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/wicket-develop



--
Living a wicket life...

Martijn Dashorst - http://www.jroller.com/page/dashorst

Wicket 1.1 is out: http://wicket.sourceforge.net/wicket-1.1

Reply via email to