Felix Meschberger wrote: > Thanks :-) I am currently working on a longer mail explaining the > changes. But here is, what I have to say to this. Ah, sorry - so I'll wait with further comments for that one.
> > >> - RequestParameter and RPMap are in a sub package while RequestPathInfo >> is not. As they both contain additional request information, I think >> they should either be all three in the core package or all three in the >> same sub package. > > How about placing all three into the o.a.s.request package ? Ok. > >> - I think we should move the request and response wrapper into a sub >> package as usually a developer does not need them. > > True, but I would like to keep them in the same place. The respective > Serlvet API wrappers are also in the same package as their interface > counterparts. Ah, yes, right - ok we can leave them there, it's not a big deal. >> - I'm not sure if "helpers" is a good name for its contents (but I'm >> currently lacking a better name) > > This name is inherited from the microsling project because I didn't have > a better name either and I didn't want to place the contents in the > o.a.s root package. :) We could put them also in the o.a.s.request package as these are special request "things". Carsten -- Carsten Ziegeler [EMAIL PROTECTED]
