Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [Wtp-wst-dev] comments about new API design in psychopath engine

Hi Jesper,
     In the new PsychoPath XPath 2.0 engine API (since ver 2.0), you've defined following two interface abstractions,

 public interface PrimitiveType extends TypeDefinition {
       ...
 }

public interface TypeDefinition {
   
     public String getNamespace();
     public String getName();
       
     public TypeDefinition getBaseType();

     public static final short DERIVATION_NONE           = 0;
     public static final short DERIVATION_EXTENSION      = 1;
     public static final short DERIVATION_RESTRICTION    = 2;
     public static final short DERIVATION_SUBSTITUTION   = 4;
     public static final short DERIVATION_UNION          = 8;
     public static final short DERIVATION_LIST           = 16;
     ...
}

The relationship between PrimitiveType and TypeDefinition in your design, looks a bit odd to me (ref, XML Schema spec [1]).

 If you intend to define this relationship, shouldn't this be the other way round, perhaps as follows,

 public interface TypeDefinition extends PrimitiveType {
     ...
 }

[1]
http://www.w3.org/TR/xmlschema-2/#primitive-vs-derived


Regards,
Mukul




From:        Mukul Gandhi/India/IBM@IBMIN
To:        wtp-wst-dev@xxxxxxxxxxx
Date:        19/04/2011 08:57 PM
Subject:        Re: [Wtp-wst-dev] comments about new API design in psychopath engine
Sent by:        wtp-wst-dev-bounces@xxxxxxxxxxx




Thanks, Jesper for the explanation. That's fair enough.



While I agree that the name is not ideal, I specifically asked for your review in early March, but got no objections?

[mukul] I apologize for not having enough time due to other commitments, during the time frame when you were expecting my review comments. If you and Dave have agreed on package naming, it's fine with me.

Regards,
Mukul


_______________________________________________
wtp-wst-dev mailing list
wtp-wst-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/wtp-wst-dev


Back to the top