Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Ensure mapped_memory_cache acts as singleton across shared objects #3306

Closed
artemp opened this issue Feb 19, 2016 · 1 comment
Closed

Ensure mapped_memory_cache acts as singleton across shared objects #3306

artemp opened this issue Feb 19, 2016 · 1 comment
Assignees
@artemp
Labels
bug c++11 tests
Milestone
Mapnik 3.0.10

Comments

@artemp
Copy link
Member

artemp commented Feb 19, 2016

Currently singleton<mapped_memory_cache, CreateStatic> gets instantiated multiple times across.
@artemp artemp added this to the Mapnik 3.0.10 milestone Feb 19, 2016
@artemp artemp self-assigned this Feb 19, 2016
artemp added a commit that referenced this issue Feb 19, 2016
@artemp
Add explicit instantiation declaration of ` singleton<mapped_memory_c…
9617555 
…ache, CreateStatic>` to avoid multiple instantiations across translation units. (ref #3306)
artemp added a commit that referenced this issue Feb 19, 2016
@artemp
clear mapped_memory_cache to ensure new '*.index` are used (MAPNIK_ME…
c4817f7 
…MORY_MAPPED_FILE) (ref #3306)
artemp added a commit that referenced this issue Feb 19, 2016
@artemp
test bogus *.index files are handled correctly (ref #3300) + indirect…
62de76a 
…ly tests #3306 via requiring `mapped_memory_cache::instance().clear()`
@artemp
Copy link
Member Author

artemp commented Feb 19, 2016

62de76a - is indirectly tests mapped_memory_cache can be used across different compilation units.

@artemp artemp closed this as completed Feb 19, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@artemp artemp

Labels
bug c++11 tests
Projects
None yet
Milestone
Mapnik 3.0.10
Development

No branches or pull requests
1 participant
@artemp