Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Temp.Add should support value optionally #6

Open
GoogleCodeExporter opened this issue Mar 14, 2015 · 0 comments
Open

Temp.Add should support value optionally #6

GoogleCodeExporter opened this issue Mar 14, 2015 · 0 comments

Comments

@GoogleCodeExporter
Copy link

Temp.Add should support value optionally

for example:

@Relation.Finish("*")
public static Temp.Add("matcher") Matcher relateWithMatcher(Matcher matcher){
    parent.add(new QName("matcher"), matcher);
}

here matcher is an abstract class. Different child elements create Matcher 
subclass instances with different
element name. We want to collect all matchers into to temp list. For this 
Temp.Add must support value argument
optionally.

workaround is:

@Relation.Finish("*")
public static void relateWithMatcher(SAXContext parent, Matcher matcher){
    parent.add(new QName("matcher"), matcher);
}

why dont we add Temp.Put also and support optional value argument for that 
also....

Original issue reported on code.google.com by [email protected] on 10 Sep 2010 at 4:39

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant