[TS-3816] Should we replace ptr_len_cmp() with memcmp() consistently?

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 7.0.0
• Component/s: Core
• Labels: newbie++
• Assignee: Tyler Stroh
• Reporter: Leif Hedstrom
• Created: 03/Aug/15 20:54
• Updated: 07/Nov/16 23:27
• Resolved: 07/Jul/16 22:59

Description

In most places, we already use memcmp(), but we have our own implementation / wrapper named ptr_len_cmp(), which is used in a few places. This seems rather inconsistent, so either we use ptr_len_cmp() consistently, or just use memcmp() across the board.

Issue Links

• links to: GitHub Pull Request #783