QuickFIX/J

new Group(int, int) should use Dictionary to decide the tag order

Details

  • Type: Improvement Improvement
  • Status: Open Open
  • Priority: Major Major
  • Resolution: Unresolved
  • Affects Version/s: None
  • Fix Version/s: None
  • Component/s: Engine
  • Labels:
    None

Description

Right now, it is:
    public Group(int field, int delim) {
        this(field, delim, new int[] { delim });
    }

But I think it should get the order info from the Dictionary to make more sense based on the spec. Thanks.

Activity

Hide
Brad Harvey added a comment - 30/Oct/08 12:18 PM

Is http://www.quickfixj.org/quickfixj/javadoc/quickfix/MessageFactory.html#create(java.lang.String,%20java.lang.String,%20int) what you're after?

Group is a basic building block - it doesn't know about Dictionary.

Show
Brad Harvey added a comment - 30/Oct/08 12:18 PM Is http://www.quickfixj.org/quickfixj/javadoc/quickfix/MessageFactory.html#create(java.lang.String,%20java.lang.String,%20int) what you're after? Group is a basic building block - it doesn't know about Dictionary.
Hide
Alvin Wang added a comment - 31/Oct/08 11:47 PM

Agree... pls close this issue.

Show
Alvin Wang added a comment - 31/Oct/08 11:47 PM Agree... pls close this issue.
Hide
Alvin Wang added a comment - 04/Nov/08 9:41 PM

i think Group(int, int) should be deprecated

Show
Alvin Wang added a comment - 04/Nov/08 9:41 PM i think Group(int, int) should be deprecated

People

Vote (0)
Watch (0)

Dates

  • Created:
    30/Oct/08 1:16 AM
    Updated:
    04/Nov/08 9:41 PM