L. Pereira
L. Pereira
Alright, thanks for the review! This should be the last round. (@jameysharp, that filter wouldn't work -- words like "keep", which are fine, would be transformed to "kep" -- but...
Pushed the requested changes. Current output here is now this one (notice the pop-up showing the mangled name):
Alright, force-pushed (sorry!) due to the formatting changes, but it should be a whole lot clearer now. I'll rebase the other PR once this one lands.
Alright, @fitzgen, pushed a new revision that should address your comments.
Rebased and force-pushed with only the change to show the function name.
Rebased (to get your patch with `prettier`/`eslint` configs) and addressed your comments.
@fitzgen Could you please merge this? (It's unlikely I'll continue working on this as this was done as part of a job, but I don't want this to bitrot.)
Force-pushed changes that: 1. Remove `LINK_LIBS` 2. Establish `DEPENDENCIES` as a more general-purpose way of declaring the dependencies of built-ins 3. Installs OpenSSL in `${CMAKE_BINARY_DIR}/deps/OpenSSL`, so it's clearer that it's...
Rebased and force-pushed with the clarifications.
(I found a way to improve this quite a bit, and am working on it. It should reduce per-bucket memory usage at the same time it makes lookups faster.)