[jira] Closed: (JELLY-97) MenuBar not shown

2004-01-07 Thread jira
Message:

   The following issue has been closed.

   Resolver: dion gillard
   Date: Wed, 7 Jan 2004 7:18 PM

Applied today
-
View the issue:

  http://jira.codehaus.org/secure/ViewIssue.jspa?key=JELLY-97


Here is an overview of the issue:
-
Key: JELLY-97
Summary: MenuBar not shown
   Type: Bug

 Status: Closed
   Priority: Minor
 Resolution: FIXED

 Original Estimate: 5 minutes
 Time Spent: Unknown
  Remaining: 5 minutes

Project: jelly
 Components: 
 taglib.swt

   Assignee: dion gillard
   Reporter: Kurt Riede

Created: Thu, 20 Nov 2003 7:19 AM
Updated: Wed, 7 Jan 2004 7:18 PM
Environment: Plattform: Win2000
Java: JDK 1.4.2

Description:
Affects version: CVS from 17.11.2003

Menubar is not attached to parent shell an thus not shown.

Reason:
Wrong signature of method attachWidgets() in class 
org.apache.commons.jelly.tags.swt.MenuTag:

What happens:
Instead of the method in the derived class MenuTag, the default method in the 
super class WidgetTag is called.

The enclosed patch fixes this problem.

Index: MenuTag.java
===
RCS file: 
/home/cvspublic/jakarta-commons/jelly/jelly-tags/swt/src/java/org/apache/commons/jelly/tags/swt/MenuTag.java,v
 retrieving revision 1.5
diff -r1.5 MenuTag.java
103c103
< protected void attachWidgets(Widget parent, Widget widget) {
---
> protected void attachWidgets(Object parent, Widget widget) {



-
JIRA INFORMATION:
This message is automatically generated by JIRA.

If you think it was sent incorrectly contact one of the administrators:
   http://jira.codehaus.org/secure/Administrators.jspa

If you want more information on JIRA, or have a bug to report see:
   http://www.atlassian.com/software/jira


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



cvs commit: jakarta-commons/jelly/jelly-tags/swt/src/java/org/apache/commons/jelly/tags/swt MenuTag.java

2004-01-07 Thread dion
dion2004/01/07 17:02:05

  Modified:jelly/jelly-tags/swt/src/java/org/apache/commons/jelly/tags/swt
MenuTag.java
  Log:
  Apply Jelly-97. Demo seems to work ok with it
  
  Revision  ChangesPath
  1.6   +1 -1  
jakarta-commons/jelly/jelly-tags/swt/src/java/org/apache/commons/jelly/tags/swt/MenuTag.java
  
  Index: MenuTag.java
  ===
  RCS file: 
/home/cvs/jakarta-commons/jelly/jelly-tags/swt/src/java/org/apache/commons/jelly/tags/swt/MenuTag.java,v
  retrieving revision 1.5
  retrieving revision 1.6
  diff -u -r1.5 -r1.6
  --- MenuTag.java  9 Oct 2003 21:21:25 -   1.5
  +++ MenuTag.java  8 Jan 2004 01:02:05 -   1.6
  @@ -100,7 +100,7 @@
* @param parent is the parent widget which is never null
* @param widget is the new child widget to be attached to the parent
*/
  -protected void attachWidgets(Widget parent, Widget widget) {
  +protected void attachWidgets(Object parent, Widget widget) {
   Menu menu = (Menu) widget;
   if (parent instanceof Decorations) {
   Decorations shell = (Decorations) parent;
  
  
  

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



[jira] Assigned: (JELLY-97) MenuBar not shown

2004-01-07 Thread jira
Message:

   The following issue has been re-assigned.

   Assignee: dion gillard (mailto:[EMAIL PROTECTED])
-
View the issue:

  http://jira.codehaus.org/secure/ViewIssue.jspa?key=JELLY-97


Here is an overview of the issue:
-
Key: JELLY-97
Summary: MenuBar not shown
   Type: Bug

 Status: Open
   Priority: Minor

 Original Estimate: 5 minutes
 Time Spent: Unknown
  Remaining: 5 minutes

Project: jelly
 Components: 
 taglib.swt

   Assignee: dion gillard
   Reporter: Kurt Riede

Created: Thu, 20 Nov 2003 7:19 AM
Updated: Wed, 7 Jan 2004 7:13 PM
Environment: Plattform: Win2000
Java: JDK 1.4.2

Description:
Affects version: CVS from 17.11.2003

Menubar is not attached to parent shell an thus not shown.

Reason:
Wrong signature of method attachWidgets() in class 
org.apache.commons.jelly.tags.swt.MenuTag:

What happens:
Instead of the method in the derived class MenuTag, the default method in the 
super class WidgetTag is called.

The enclosed patch fixes this problem.

Index: MenuTag.java
===
RCS file: 
/home/cvspublic/jakarta-commons/jelly/jelly-tags/swt/src/java/org/apache/commons/jelly/tags/swt/MenuTag.java,v
 retrieving revision 1.5
diff -r1.5 MenuTag.java
103c103
< protected void attachWidgets(Widget parent, Widget widget) {
---
> protected void attachWidgets(Object parent, Widget widget) {



-
JIRA INFORMATION:
This message is automatically generated by JIRA.

If you think it was sent incorrectly contact one of the administrators:
   http://jira.codehaus.org/secure/Administrators.jspa

If you want more information on JIRA, or have a bug to report see:
   http://www.atlassian.com/software/jira


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



Re: [digester] mavenization of documentation build

2004-01-07 Thread Simon Kitching
On Thu, 2004-01-08 at 10:56, robert burrell donkin wrote:

> at first glance it seems that pretty much everything is there but i'd 
> like to double check before it's declared official. i like to ask first 
> just in case any other people have any strong feelings about not 
> mavenizing digester. (now would be a good time to speak up if you have, 
> BTW)

I'm happy to use Maven for Digester.



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



cvs commit: jakarta-commons/beanutils/src/java/org/apache/commons/beanutils package.html

2004-01-07 Thread rdonkin
rdonkin 2004/01/07 14:01:09

  Modified:beanutils/src/java/org/apache/commons/beanutils package.html
  Log:
  Added entry to FAQ about ordering bean comparator.
  
  Revision  ChangesPath
  1.18  +24 -0 
jakarta-commons/beanutils/src/java/org/apache/commons/beanutils/package.html
  
  Index: package.html
  ===
  RCS file: 
/home/cvs/jakarta-commons/beanutils/src/java/org/apache/commons/beanutils/package.html,v
  retrieving revision 1.17
  retrieving revision 1.18
  diff -u -r1.17 -r1.18
  --- package.html  17 Sep 2003 20:23:22 -  1.17
  +++ package.html  7 Jan 2004 22:01:09 -   1.18
  @@ -53,6 +53,7 @@
   Frequently Asked Questions
   
   Why Can't BeanUtils Find My Method?
  +How Do I Set The BeanComparator Order To Be 
Ascending/Descending?
   
   
   
  @@ -808,5 +809,28 @@
   create your own BeanInfo.
   
   
  +
  +How Do I Set The BeanComparator Order To Be Ascending/Descending?
  +
  +BeanComparator relies on an internal Comparator to perform the actual 
  +comparisions. By default, 
  +org.apache.commons.collections.comparators.ComparableComparator 
  +is used which imposes a natural order. If you want to change the order, 
  +then a custom Comparator should be created and passed into the 
  +appropriate constructor.
  +
  +
  +For example:
  +
  +
  +import org.apache.commons.collections.comparators.ComparableComparator;
  +import org.apache.commons.collections.comparators.ReverseComparator;
  +import org.apache.commons.beanutils.BeanComparator;
  +...
  +BeanComparator reversedNaturalOrderBeanComparator
  += new BeanComparator("propertyName", new ReverseComparator(new 
ComparableComparator()));
  +Collections.sort(myList, reversedNaturalOrderBeanComparator);
  +...
  +
   
   
  
  
  

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



Re: [digester] mavenization of documentation build

2004-01-07 Thread robert burrell donkin
On 6 Jan 2004, at 22:45, __matthewHawthorne wrote:

robert burrell donkin wrote:
there has been talk about an improved, common documentation build for 
jakarta commons. IIRC the consensus settled on maven. it was 
suggested that those components already with mavenized builds switch 
to using these for documentation.
i'd be happy to look at updating the digester maven build and porting 
the existing site to maven. one reason why i'd like to do this is 
that it should be possible to generate html from the source in the 
examples (which i think would be really cool :)


http://jakarta.apache.org/commons/digester already looks Mavenized.
yep.

Are you speaking of http://jakarta.apache.org/commons/digester.html?  
It
seems to be a pretty simple conversion.
yep. (but never said it'd be difficult ;)

at first glance it seems that pretty much everything is there but i'd 
like to double check before it's declared official. i like to ask first 
just in case any other people have any strong feelings about not 
mavenizing digester. (now would be a good time to speak up if you have, 
BTW)

HTML source examples are always helpful, good idea.
the examples are pretty cool already but i think folks are more likely 
to discover them if they're on the website.

- robert

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


Re: [BeanUtils] WrapDynaBean Enhacement Request

2004-01-07 Thread robert burrell donkin
On 7 Jan 2004, at 03:27, Craig R. McClanahan wrote:
Quoting robert burrell donkin <[EMAIL PROTECTED]>:


3 seems to me to be a symptom of a bigger issue (which has been known
for some time). the exception handling in beanutils is painful and
confusing to many users. i'd be very reluctant to break backwards
compatibility (even for symantics) and i suspect that the proposed
patch does. we've talked before about the possibility of factoring out
the exception handling and possible it might be better to fix this
rather than the symptom.  so maybe a little more talk and thought is
needed about this one.
I'm on general principles opposed to breaking backward compatibility 
on method
names -- but I don't have any problem with efficiency improvements, or
usability improvements (i.e. error messages that point you at what the 
real
issue is).  I haven't had time to look at your patch yet (slightly 
busy with
JavaServer Faces at the moment ;-), but agree with Robert's general 
point that
an overall review of how BeanUtils is put together as a whole is 
probably
overdue.  Indeed, that sort of thing would make sense in the context 
of a
BeanUtils 2.0 type discussion, perhaps having 2.x releases while we 
continue
bugfixes on the 1.x track and keep compatibility there.

BeanUtils is a very widely used package, so we have to be even more 
careful than
is typical in j-c about compatibility.  But it's also time to do some
innovation again, and these sorts of discussions might be a good 
kickoff for
that.
+1

i have an idea that there are a number of much requested features that 
it might be possible to add without breaking backwards compatibility. 
there will be a price to pay in terms of introducing more strategy 
plug-ins and greater complexity but i believe that the advantages of 
being able to drop in more sophisticated functionality would be great.

now the main package has been beanified, it might be possible to allow 
users to tweak existing applications without being forced to upgrade.

exception handling is one that's been talked about frequently. many 
users find the current exception handling strategy confusing. there has 
been talk about whether it'd be possible to factor out. i'm not sure 
whether this would be possible or whether it'd solve niall's problems 
but maybe it's something we might think about.

- robert

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


Re: [BeanUtils] WrapDynaBean Enhacement Request

2004-01-07 Thread robert burrell donkin
On 7 Jan 2004, at 05:44, Niall Pemberton wrote:

My problem is I built a framework to just generate, process and spit 
out
DynaBeans and then got it to also do POJ beans afterwards when I 
discovered
your "Wrapped" DynaBeans .

1) WrapDynaClass.newInstance()
The framework generates DynaBeans (from Excel files). I am storing the
DynaClasses and using newInstance() to generate DynaBeans as it needs 
them.
It just made adapting my framework to generate ordinary JavaBeans via
"Wrapped" dynabeans v.easy with WrapDynaClass.newInstance() working 
the same
as other DynaClasses. I agree with you, if I hadn't built it 
initiallly just
for DynaBeans I could have done it differently.

Also if WrapDyanClass doesn't implement the newInstance() method then 
it
doesn't fulfill its contract as a DynaClass type - shouldn't it do it
anyway, even if there are other ways to generate the WrapDynaBean -
otherwise it has to be catered for as special case.
this seems like a reasonable use case. i'm in favour of adding this 
method (with some more documentation giving advice about the most 
typical use case). i'll probably go ahead and add this one unless 
sometime soonish.

2) WrapDynaBean.getInstance()
The framework runs out of the box to generate DynaBeans from Excel
spreadsheets using XML configuration. If I can get the original 
JavaBean out
of the Wrapper, I don't have to worry about modifying my existing 
framework.
I think this is a good use case - have a system where you can plug in a
Factory to generate whatever flavor of DynaBeans are required, process 
them
and then spit them out to a downstreams system (which has no reference 
to
those POJ Beans) which can then "UnWrap" them and use them as it likes.
being able to obtain a reference to the wrapped bean seems reasonable 
to me. again, this kind of thing is probably not appropriate for the 
most common use cases but would be useful for some kinds of design. 
i'll probably go ahead and add it (with some appropriate documentation) 
sometime soonish.

- robert

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


Re: [net] 1.2 release (Re: [net] checked in parser factory implementation)

2004-01-07 Thread steve cohen
On Wednesday 07 January 2004 11:44 am, Daniel F. Savarese wrote:
> In message <[EMAIL PROTECTED]>, Jeffrey D. Brekke writes:
> >[Alternatives to VMS Parser/Version issue]
> >
> >Another alternative is to create another parser, creating two VMS
> >parsers, potentially sub-classing one VMS parser to avoid code
> >duplication.  A specialized VMS parser that will filter off multiple
> >versions.  This solves the contract problems with the parsers and
>
> I was about to say "Eureka!, that's the right solution." as far
> as the specific VMS parser case goes, but there's still the problem
> of how to make it filter off multiple versions when called using
> readNextEntry and parseFTPEntry.  Unless I'm missing something,
> we still have to support some hook for the postfiltering.  Nonetheless,
> splitting the VMS parsing functionality into two separate classes
> (one derived from the other) is cleaner than using the versioning
> property.

I think we're all in agreement here.  The subclass is cleanest the way to go.
(See, Jeffrey, that wasn't so hard).

I also agree that there is a need for postfiltering (to solve the "most recent 
version only" problem), although, actually, I think prefiltering out the 
dupes may be the way to go.  Either way, some kind of hook will be necessary, 
one that's a no-op in the default case.  If time gets to be a factor we can 
leave this as a known issue for later.  No one's actually complained about it 
other than me, from reading the code.

>
> >[FTPClient API coherence]
> >
> >On the point of the FTPClient api, I was under the impression also
> >that we were leaning toward a FTPFileList as the norm, and away from
> >the arrays.  Maybe now that we're 1.2 bound, we can just return List
> >and have FTPFileList implement the List interface ( and Iterator
> >interface, opening up the possibility of using commons/collections
> >filter iterator or other collection utilities )?
>
> I see Steven and you have made further comments in the thread about
> this.  I'm in favor of whatever provides sufficient flexibility so
> that API users can customize behavior without requiring us to
> shoehorn application-specific functionality into the library.
> I'm not sure about having FTPFileList implement List, or perhaps
> Collection, but I haven't sorted out my thoughts.  There could
> be considerable advantages.  Also, one of the possibilities I
> threw out was having void listFiles(FTPFileList, ...), where the
> results are returned in the FTPFileList, which would require at
> least clear() and possibly add() methods depending on the
> implementation.
>
> >I guess we're spiraling out of control here with ideas ( not
> >necessarily a bad thing ).  Just not sure how to rein us in ;)
>
> Although I don't want to put off decisions, we can always take
> a baby step to resolve the immediate VMS entry parser issue and
> take some more time to figure out the rest.  That is, unless
> they are inextricably tied ...  We've got several options for
> allowing ant to grab a VMS entry parser with version filtering.
> I think we've agreed that splitting out the functionality into
> a subclass is the way to go.  But we still need a way to implement
> the version filtering transparently without depending on parseFileList.
>
> daniel
>
>

Frankly I think we could have this coded by the weekend if not sooner.


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



Re: [net] 1.2 release (Re: [net] checked in parser factory implementation)

2004-01-07 Thread steve cohen
On Wednesday 07 January 2004 11:14 am, Daniel F. Savarese wrote:
> In message <[EMAIL PROTECTED]>, steve cohen writes:
> >I keep coming back to the ant use case and how we'd handle it there.  I
> >suppose we could add yet another parameter to the ant  task to handle
> >this odd case, but I'd rather not.  I'm still not happy with this but I
> > don't have a better suggestion yet.
>
> Okay, so am I correct that in ant you can explictly define the parser key?
> If that's the case then one can implement a factory that recognizes a
> special parser key (or add it to the default factory), like VMSV that
> will create a VMS entry parser which handles versioning.  That is, assuming
> the ant task calls listFiles(String, String), which is what I understood
> to be the reason you gave to keep the method public when Jeffrey suggested
> it not be public.
>
> daniel
>
>
>
> -
> To unsubscribe, e-mail: [EMAIL PROTECTED]
> For additional commands, e-mail: [EMAIL PROTECTED]

Exactly so.  This will work.

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



RE: [net] 1.2 release (Re: [net] checked in parser factory implementation)

2004-01-07 Thread Noel J. Bergman
Daniel F. Savarese wrote:
> I also forgot to add that there's at least one pending code
submission/patch
> that we need to review and include before a 1.2 release.  I believe it was
> the NTP/SNTP functionality submitted by Jason Matthews.

Correct.  On September 14, and corrected on October 8.


http://nagoya.apache.org/eyebrowse/[EMAIL PROTECTED]
he.org&msgNo=33993


http://nagoya.apache.org/eyebrowse/[EMAIL PROTECTED]
he.org&msgNo=35816

> We may need a signed CLA before we can include it should it pass muster.

This is an area I find confusing, as well, but AIUI to date we mostly
require a CLA only from a committer.  If the codebase is significant (a grey
area), one could ask for a software grant, as we have done in some cases.
The revised ASF licenses are supposed to help clear up questions related to
contributions.  But Jason did post it to the mailing list for the specific
purpose of having it incorporated, and he applied the ASF License to each
file.

--- Noel


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



Re: [net] 1.2 release (Re: [net] checked in parser factory implementation)

2004-01-07 Thread Daniel F. Savarese

In message <[EMAIL PROTECTED]>, Jeffrey D. Brekke writes:
>[Alternatives to VMS Parser/Version issue]
>
>Another alternative is to create another parser, creating two VMS
>parsers, potentially sub-classing one VMS parser to avoid code
>duplication.  A specialized VMS parser that will filter off multiple
>versions.  This solves the contract problems with the parsers and

I was about to say "Eureka!, that's the right solution." as far
as the specific VMS parser case goes, but there's still the problem
of how to make it filter off multiple versions when called using
readNextEntry and parseFTPEntry.  Unless I'm missing something,
we still have to support some hook for the postfiltering.  Nonetheless,
splitting the VMS parsing functionality into two separate classes
(one derived from the other) is cleaner than using the versioning
property.

>[FTPClient API coherence]
>
>On the point of the FTPClient api, I was under the impression also
>that we were leaning toward a FTPFileList as the norm, and away from
>the arrays.  Maybe now that we're 1.2 bound, we can just return List
>and have FTPFileList implement the List interface ( and Iterator
>interface, opening up the possibility of using commons/collections
>filter iterator or other collection utilities )?  

I see Steven and you have made further comments in the thread about
this.  I'm in favor of whatever provides sufficient flexibility so
that API users can customize behavior without requiring us to
shoehorn application-specific functionality into the library.
I'm not sure about having FTPFileList implement List, or perhaps
Collection, but I haven't sorted out my thoughts.  There could
be considerable advantages.  Also, one of the possibilities I
threw out was having void listFiles(FTPFileList, ...), where the
results are returned in the FTPFileList, which would require at
least clear() and possibly add() methods depending on the
implementation.

>I guess we're spiraling out of control here with ideas ( not
>necessarily a bad thing ).  Just not sure how to rein us in ;)

Although I don't want to put off decisions, we can always take
a baby step to resolve the immediate VMS entry parser issue and
take some more time to figure out the rest.  That is, unless
they are inextricably tied ...  We've got several options for
allowing ant to grab a VMS entry parser with version filtering.
I think we've agreed that splitting out the functionality into
a subclass is the way to go.  But we still need a way to implement
the version filtering transparently without depending on parseFileList.

daniel



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



Re: [net] 1.2 release (Re: [net] checked in parser factory implementation)

2004-01-07 Thread Daniel F. Savarese

In message <[EMAIL PROTECTED]>, steve cohen writes:
>I keep coming back to the ant use case and how we'd handle it there.  I 
>suppose we could add yet another parameter to the ant  task to handle 
>this odd case, but I'd rather not.  I'm still not happy with this but I don't 
>have a better suggestion yet.

Okay, so am I correct that in ant you can explictly define the parser key?
If that's the case then one can implement a factory that recognizes a
special parser key (or add it to the default factory), like VMSV that
will create a VMS entry parser which handles versioning.  That is, assuming
the ant task calls listFiles(String, String), which is what I understood
to be the reason you gave to keep the method public when Jeffrey suggested
it not be public.

daniel



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



[cache] Common abstraction layer

2004-01-07 Thread Emmanuel Bourg
Hi, I have implemented what could be the basis for a common abstraction 
layer for cache implementations as discussed previously :

http://www.mail-archive.com/[EMAIL PROTECTED]/msg31432.html
http://www.mail-archive.com/[EMAIL PROTECTED]/msg32363.html
The structure of this implementation is similar to commons-logging. The 
CacheFactory class has been adapted from the LogFactory class.

Let me know if this looks ok :)

Emmanuel Bourg



cache.zip
Description: Zip archive


smime.p7s
Description: S/MIME Cryptographic Signature


Re: [net] 1.2 release (Re: [net] checked in parser factory implementation)

2004-01-07 Thread Jeffrey D. Brekke
> On Wed, 07 Jan 2004 05:44:36 -0600, steve cohen <[EMAIL PROTECTED]> said:

> On Tuesday 06 January 2004 11:04 pm, Jeffrey D. Brekke wrote:
>> Steve/Daniel,
>> 
>> I'm responding to snippets from several messages in this one
>> message, sorry if it's confusing...
>> 
>> [Iterator idea]
>> 
>> When I suggested the filtering iterator I was thinking of something
>> really simple, like an additional class or example of how one could
>> filter off multiple versions when iterating and leaving the VMS
>> parser simple, just parsing the list with versions intact.

> This sounds interesting but I don't really understand how it would
> work.  Do you have some example from some other system in mind?

[SNIPPED]

No, but I was just thinking that we'd use Commons/Collections
FilterIterator or some such and provide either the implementation or
an example of how one could do it.  A FilterIterator that would
basically just suppress multiple versions.

I haven't done any more than look into javadoc really.  If I get time
I'll try it out.

>>  [Alternatives to VMS Parser/Version issue]
>> 
>> Another alternative is to create another parser, creating two VMS
>> parsers, potentially sub-classing one VMS parser to avoid code
>> duplication.  A specialized VMS parser that will filter off
>> multiple versions.  This solves the contract problems with the
>> parsers and keeps the classes following SRP. If someone doesn't
>> like the VMS parser our factory defaults to for auto-detection (
>> we'll have to decide on one of them ), they could write their own
>> factory ( or we could write a property-backed factory for easy
>> customization or something ).

> I was thinking along these lines too.  I didn't pursue it because it
> seemed ugly, but is it?  I'm not so sure it is.  VMS with versioning
> is really a very significant difference from VMS without versioning,
> and yet similar enough that the IS-A relation is a good fit.  It's
> certainly easier to do this than to do what Daniel and I were
> talking about, and it's conceptually defensible too.  We don't need
> to change the factory for this either - just like the Enhanced Unix,
> this one wouldn't be available to auto-detection, but accessible via
> key or classname and manual coding.
>>  [FTPClient API coherence]
>> 
>> On the point of the FTPClient api, I was under the impression also
>> that we were leaning toward a FTPFileList as the norm, and away
>> from the arrays.  Maybe now that we're 1.2 bound, we can just
>> return List and have FTPFileList implement the List interface ( and
>> Iterator interface, opening up the possibility of using
>> commons/collections filter iterator or other collection utilities
>> )?

> I'm not sure I agree here.  FTPFileList (despite its name) is not a
> list of FTPFiles.  It is a wrapper around vector of raw FTP entries
> designed to implement Daniel's old suggestion that a quick read of
> the list be separated in time from the creation of more expensive
> FTPFile objects, which would be created as needed when iterating the
> list in a paged fashion, or possibly by a different thread.

> This is not to say that there might not be a good reason to make
> listFiles() methods return Lists instead of arrays, but that's
> different from FTPFileList.
>>  I guess we're spiraling out of control here with ideas ( not
>> necessarily a bad thing ).  Just not sure how to rein us in ;)

> If we're still at this point by next week, then you should worry.
> :-)

>>  > On Tue, 06 Jan 2004 19:47:15 -0600, steve cohen >
>> <[EMAIL PROTECTED]> said:
>> >
>> > On Tuesday 06 January 2004 07:08 pm, Daniel F. Savarese wrote:
>> >> In message <[EMAIL PROTECTED]>, steve
>> cohen >> writes: >Almost right, Daniel.  I think it filters out
>> dupes when >> versioning is > turned on.
>> >>
>> >> I thought that's what you said before, but I saw if (versioning)
>> { >> files = super.parseFileList(listingStream); } else { in >>
>> VMSFTPEntryParser.parseFileList.  Is that an error?  Should it be
>> >> if (!versioning) or do I have the meaning of the versioning >>
>> variable mixed up?  Just wondering if we found a bug.
>> >
>> > You're right.  I'm sorry.  You read what the code said.  I "read"
>> > what I thought the code should be saying.  It does seem >
>> counter-intuitive the way it is, but maybe there's a way to >
>> understand it that I don't have, by which it makes sense.  I've >
>> explicitly added Stephane Este-Gracias (who wrote this code) to
>> this > thread for his opinion.  Stephane, if you see this, please
>> weigh in > on this.
>> >
>> >> >Actually, I like your suggestion.  The iterator seems the right
>> >>
>> >> place to > do it.
>> >>
>> >> As you know by now from my subsequent email, I have yet another
>> >> suggestion :)
>> >
>> > Which I've already responded to so will say no more here.
>> >
>> >> >Here's another problem, though, in our system.  How do you turn
>> >>
>> >> versioning > on in the auto-detect scenario?  There's no hook in
>> >>

Re: [betwixt] Status of Betwixt

2004-01-07 Thread Martin van den Bemt
Btw I am going to use commons-sql (at least a choice for now) for my
work, which depends on betwixt, so with a bit of luck I can do some
betwixt during office hours...

Mvgr,
Martin

On Tue, 2004-01-06 at 23:31, robert burrell donkin wrote:
> On 6 Jan 2004, at 02:50, Ryan Hoegg wrote:
> 
> > I am a happy betwixt user and would be happy to take a more active 
> > role sometime in late January.
> 
> cool. more hands, eyes and brains would be great :)
> 
> > What are your concerns with betwixt as it is now?
> 
> too many to list quickly. it took me a long time to understand a little 
> about what a start-from-java mapper needs to do. now i think i have an 
> idea, the current betwixt internal structure isn't strong and simple 
> enough to support it.
> 
> there are some list of planned features on the todo list in the 
> documentation.
> 
> the refactoring i've been (slowly) doing is a complete rewrite (again) 
> of the xml reading code.
> 
> > Perhaps a little road map?
> 
> i'd suggest that we start off as martin says: i'll tidy up my 
> refactored code and create a branch this weekend. we'll get together 
> for a chat on IRC sometime next week or weekend.
> 
> i know that some people want (and possibly need) a proper (non-alpha) 
> release. the code in cvs head is probably good enough for a 0.1 or 0.5 
> release but some other committer would need to manage it. this could be 
> done as soon as convenient. (the more advanced refactored stuff would 
> be on the branch.)
> 
> > Do we have a wiki?
> 
> probably :)
> 
> but IIRC the apache wiki's (does anyone know more?) are being 
> restructured so maybe it'd be a good idea to hold off until the dust 
> settles
> 
> - robert
> 
> 
> -
> To unsubscribe, e-mail: [EMAIL PROTECTED]
> For additional commands, e-mail: [EMAIL PROTECTED]
-- 
Mvgr,
Martin


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



RE: [collections] Version 3.0 - The final stretch

2004-01-07 Thread scolebourne
Neither of these yet exist in released versions of beanutils/configuration, so they 
can't be deprecated yet.

Stephen

>  from:Henri Yandell <[EMAIL PROTECTED]>
> Weren't ExtendedProperties and BeanMap to be removed?
> 
> Or was the decision made to keep them?


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



Re: [net] 1.2 release (Re: [net] checked in parser factory implementation)

2004-01-07 Thread steve cohen
On Tuesday 06 January 2004 11:04 pm, Jeffrey D. Brekke wrote:
> Steve/Daniel,
>
> I'm responding to snippets from several messages in this one message,
> sorry if it's confusing...
>
> [Iterator idea]
>
> When I suggested the filtering iterator I was thinking of something
> really simple, like an additional class or example of how one could
> filter off multiple versions when iterating and leaving the VMS parser
> simple, just parsing the list with versions intact.

This sounds interesting but I don't really understand how it would work.  Do 
you have some example from some other system in mind?

>
> [Alternatives to VMS Parser/Version issue]
>
> Another alternative is to create another parser, creating two VMS
> parsers, potentially sub-classing one VMS parser to avoid code
> duplication.  A specialized VMS parser that will filter off multiple
> versions.  This solves the contract problems with the parsers and
> keeps the classes following SRP. If someone doesn't like the VMS parser
> our factory defaults to for auto-detection ( we'll have to decide on
> one of them ), they could write their own factory ( or we could write
> a property-backed factory for easy customization or something ).

I was thinking along these lines too.  I didn't pursue it because it seemed 
ugly, but is it?  I'm not so sure it is.  VMS with versioning is really a 
very significant difference from VMS without versioning, and yet similar 
enough that the IS-A relation is a good fit.  It's certainly easier to do 
this than to do what Daniel and I were talking about, and it's conceptually 
defensible too.  We don't need to change the factory for this either - just 
like the Enhanced Unix, this one wouldn't be available to auto-detection, but 
accessible via key or classname and manual coding.
>
> [FTPClient API coherence]
>
> On the point of the FTPClient api, I was under the impression also
> that we were leaning toward a FTPFileList as the norm, and away from
> the arrays.  Maybe now that we're 1.2 bound, we can just return List
> and have FTPFileList implement the List interface ( and Iterator
> interface, opening up the possibility of using commons/collections
> filter iterator or other collection utilities )?

I'm not sure I agree here.  FTPFileList (despite its name) is not a list of 
FTPFiles.  It is a wrapper around vector of raw FTP entries designed to 
implement Daniel's old suggestion that a quick read of the list be separated 
in time from the creation of more expensive FTPFile objects, which would be 
created as needed when iterating the list in a paged fashion, or possibly by 
a different thread.

This is not to say that there might not be a good reason to make listFiles() 
methods return Lists instead of arrays, but that's different from 
FTPFileList.
>
> I guess we're spiraling out of control here with ideas ( not
> necessarily a bad thing ).  Just not sure how to rein us in ;)

If we're still at this point by next week, then you should worry.
:-)

>
> > On Tue, 06 Jan 2004 19:47:15 -0600, steve cohen
> > <[EMAIL PROTECTED]> said:
> >
> > On Tuesday 06 January 2004 07:08 pm, Daniel F. Savarese wrote:
> >> In message <[EMAIL PROTECTED]>, steve cohen
> >> writes: >Almost right, Daniel.  I think it filters out dupes when
> >> versioning is > turned on.
> >>
> >> I thought that's what you said before, but I saw if (versioning) {
> >> files = super.parseFileList(listingStream); } else { in
> >> VMSFTPEntryParser.parseFileList.  Is that an error?  Should it be
> >> if (!versioning) or do I have the meaning of the versioning
> >> variable mixed up?  Just wondering if we found a bug.
> >
> > You're right.  I'm sorry.  You read what the code said.  I "read"
> > what I thought the code should be saying.  It does seem
> > counter-intuitive the way it is, but maybe there's a way to
> > understand it that I don't have, by which it makes sense.  I've
> > explicitly added Stephane Este-Gracias (who wrote this code) to this
> > thread for his opinion.  Stephane, if you see this, please weigh in
> > on this.
> >
> >>  >Actually, I like your suggestion.  The iterator seems the right
> >>
> >> place to > do it.
> >>
> >> As you know by now from my subsequent email, I have yet another
> >> suggestion :)
> >
> > Which I've already responded to so will say no more here.
> >
> >>  >Here's another problem, though, in our system.  How do you turn
> >>
> >> versioning > on in the auto-detect scenario?  There's no hook in
> >> listFiles() for doing > so.
> >>
> >> I would say that's where the FTPFileEntryParserFactory comes in.
> >> If someone wants VMSFTPEntryParsers with versioning turned on, they
> >> can implement a factory that returns them.  We could add a
> >> setVMSVersioning(boolean) method to
> >> DefaultFTPFileEntryParserFactory and save users the trouble.
> >> They'd have to do the following: FTPClient ftp = new FTPClient();
> >> DefaultFTPFileEntryParserFactory factory = new
> >> DefaultFTPFileEntryParserFactory(); factory.s