-
Notifications
You must be signed in to change notification settings - Fork 350
Use released abseil-cpp, not internal copy #168
Copy link
Copy link
Closed
Description
I was waiting until bytes_used was in abseil-cpp, but I'm not sure when that will happen, so I might just fake it.
s2geometry/src/s2/mutable_s2shape_index.cc
Line 1517 in a4dddf4
| size += cell_map_.bytes_used() - sizeof(cell_map_); |
Using a real abseil-cpp does raise some questions. Should it be a submodule? Or do we require that it's installed somewhere? abseil is hostile to being installed (only LTS releases can be installed in standard locations like /usr/local) because it doesn't want to guarantee stability.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels