Skip to content
This repository has been archived by the owner on Feb 8, 2019. It is now read-only.

fixed JaxpAttributeAssignment constructor #5

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

duanshiqiang
Copy link

The sequence of the parameters passed to the constructor of class JaxpAttributeAssignment was wrong.

AttributeValue<?> attributeValueIn) {
super(attributeIdIn, categoryIdIn, issuerIn, attributeValueIn);
super(categoryIdIn, attributeIdIn, issuerIn, attributeValueIn);
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this looks like from documentation, StdMutableAttributeAssignment(Identifier categoryIn, Identifier attributeIdIn, java.lang.String issuerIn, AttributeValue<?> attributeValueIn)

@jainh
Copy link

jainh commented Oct 20, 2016

Looks good to me.

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

Successfully merging this pull request may close these issues.

2 participants