gvpack: replace CDT pair dictionaries with 'std::multiset'
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Thu, 22 Sep 2022 04:04:35 +0000 (21:04 -0700)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Mon, 3 Oct 2022 00:30:52 +0000 (17:30 -0700)
commit294ea602f9129ea2fb683d78b6008dbb462e0820
treee490d37e613a15a43e4ace521c5c09c76327a351
parent3c1b54e764661a83cf1974da840cc09c8b039266
gvpack: replace CDT pair dictionaries with 'std::multiset'

This is intended to be a non-functional change with a number of benefits:

  1. This is C++ code, so no need for custom CDT data structures. The STL
     equivalents are likely more optimized, better understood by the compiler,
     and more familiar to human readers.

  2. We avoid the lib/common allocation routines (`NEW` macro and friends). Now
     allocation failures naturally propagate outwards as `std::bad_alloc`
     exceptions without any custom handling.

  3. Related to (2), we no longer need to manually free any of these objects.
     RAII takes care of all of this.
cmd/tools/gvpack.cpp