I have changed my mind, and gone ahead and submitted the patch to Gerrit. I'd
be very interested in your and anyone else's feedback regarding the
approach. It compiles fine, but I still need to figure out how to test it. I
also still have reservations about mixing C++ and C memory allocation.
Than
I took a crack at it, but it resulted in too much mixing of C and C++ style
memory allocation, which I'm not comfortable intermingling. The overwhelming
majority of the original is implemented in C anyway, so I'm not sure it's
worth me replacing just one facet of the code with C++. Perhaps one day
On Fri, 2014-05-16 at 10:46 -0700, Chris Laplante wrote:
> Would it be possible to remove vcl/source/fontsubset/list.cxx and replace its
> usage with std::list? The only place I see it being used is in ttcr.cxx
> (same folder).
Looks like it should be reasonably straightforward to do that.
C.
_
Would it be possible to remove vcl/source/fontsubset/list.cxx and replace its
usage with std::list? The only place I see it being used is in ttcr.cxx
(same folder).
It is hard to tell because most of the files in that directory appear to be
C code hiding in .cxx files.
Chris
--
View this mess
