> and please rename arg0 to "target" :) It's a good idea to attach
> Wicket sources to your IDE (e.g. mvn -DdownloadSources=true eclipse:eclipse)
> because then this tends to happen automatically.

I have been thinking of attaching the sources... I found some example
of adding all sources of all packages in the repository. Is there a
way to limit this to e.g., specific packages? This way I can lazy-load
the sources whenever needed. And does it require some additional setup
from Eclipse (e.g., pom.xml/project properties) or will it
automatically find the existing sources from maven repository?

> > >     add(new FeedbackPanel("feedback"));
> Change this to
>     add(new FeedbackPanel("feedback").setOutputPlaceHolderTag(true));

I will try these and comment.
> > >   /**
> > >    * This method TODO
> > >    */
> ...and it would be a good idea to cleanup code formatting
> and remove these comments...

I prefer to comment all non-private members and the template gives me a TODO...

> > >     try {
> > >       Thread.sleep(5000);
> > >     } catch (InterruptedException e) {
> > >       e.printStackTrace();
> > >     }
> ...and fix the exception handling, throw new RuntimeException(e);

Relax, it's just mock-up-code, but yes, I completely agree with you on this ;)

**
Martin

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to