[PIVOT-648] useless ResultList constructors due to ignored argument

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 1.5.1
• Fix Version/s: 1.5.2, 2.0
• Component/s: core-sql
• Assignee: Unassigned
• Reporter: Andy Heninger
• Created: 27/Sep/10 21:59
• Updated: 27/Sep/10 22:29
• Resolved: 27/Sep/10 22:29

Description

two constructors have identical bodies to other constructors due to ignored argument:
public ResultList(ResultSet resultSet, Field... fields)

{ this(resultSet, false, new ArrayList<Field>(fields)); }
vs
public ResultList(ResultSet resultSet, boolean includeNullValues, Field... fields) { this(resultSet, false , new ArrayList<Field>(fields)); }
and
public ResultList(ResultSet resultSet, Sequence<Field> fields) { this(resultSet, false, new ArrayList<Field>(fields)); }

vs
public ResultList(ResultSet resultSet, boolean includeNullValues, Sequence<Field> fields)

{ this(resultSet, false , new ArrayList<Field>(fields)); }