Skip to content
This repository has been archived by the owner on Aug 8, 2023. It is now read-only.

[core] Assert fires in LayerFactory::createBucket #15636

Closed
julianrex opened this issue Sep 16, 2019 · 1 comment · Fixed by #15644
Closed

[core] Assert fires in LayerFactory::createBucket #15636

julianrex opened this issue Sep 16, 2019 · 1 comment · Fixed by #15644
Assignees
Labels
beta blocker Blocks the next beta release Core The cross-platform C++ core, aka mbgl high priority iOS Mapbox Maps SDK for iOS

Comments

@julianrex
Copy link
Contributor

julianrex commented Sep 16, 2019

As of 40b0cb7, when running the iOS Integration Test Harness tests, specifically testForRaisingExceptionsOnStaleLayerObject, there's an assert in:

std::unique_ptr<Bucket> LayerFactory::createBucket(const BucketParameters&, const std::vector<Immutable<style::LayerProperties>>&) noexcept {
assert(false);
return nullptr;
}

for a factory of type RasterStyleLayerPeerFactory. Missing function?

@tmpsantos suggested this may be a regression introduced in #15308

/cc @tobrun

@julianrex julianrex changed the title [core] [core] Assert fires in LayerFactory::createBucket Sep 16, 2019
@julianrex julianrex added the beta blocker Blocks the next beta release label Sep 16, 2019
@pozdnyakov
Copy link
Contributor

The reason is that we allow adding layers incompatible with sources. In the test a raster layer is added to a vector source.

@julianrex julianrex added Core The cross-platform C++ core, aka mbgl iOS Mapbox Maps SDK for iOS labels Sep 17, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
beta blocker Blocks the next beta release Core The cross-platform C++ core, aka mbgl high priority iOS Mapbox Maps SDK for iOS
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants