[ORC-1348] TimezoneImpl constructor should pass std::vector<> & instead of std::vector<>

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 1.9.0
• Fix Version/s: 1.9.0
• Component/s: C++
• Assignee: Junwang Zhao
• Reporter: Junwang Zhao
• Created: 08/Jan/23 14:16
• Updated: 09/Jan/23 23:24
• Resolved: 09/Jan/23 23:23

Description

TimezoneImpl(const std::string& name, const std::vector<unsigned char> bytes) should
change to TimezoneImpl(const std::string& name, const std::vector<unsigned char>& bytes) to reduce copy constructor of vector of unsigned char.

Issue Links

• links to: GitHub Pull Request #1358