| Commit message (Collapse) | Author | Age | Files | Lines |
|---|
| |
|
|
| |
As reported by clang-tidy-8.
|
| |
|
|
| |
As reported by clang-tidy-8.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* [build] Fix integer overflow runtime error for core part
Temporarily remove circle ci UBSAN build precondition
* [build] Enable all of the ubsans
[build] Check runtime error
[build] Update UBSAN_OPTION
* [build] Add UBSAN blacklist
[build] Ignore system libraries
[build] Ignore vendor library
* [build] Fix implicit conversion runtime error in core
* [build] Fix division by zero runtime error
* [build] Add unfixed error to ubsan blacklist
* [build] Make UBSAN halt on error
Revert "Temporary remove build precondition"
* [build] Fix division by zero error
* [build] Make UBSAN officially work without FIXME prefix
* [build] Fix implicit conversion from int64_t to uint64_t
* [build] Rename style test json file name
* Address review findings
|
| | |
|
| | |
|
| |
|
|
| |
As reported by clang-tidy-8.
|
| |
|
|
| |
As reported by clang-tidy-8.
|
| |
|
|
| |
As reported by clang-tidy-8.
|
| | |
|
| |
|
|
| |
As reported by clang-tidy-8.
|
| |
|
|
| |
As reported by clang-tidy-8.
|
| | |
|
| | |
|
| |
|
|
| |
operator
|
| | |
|
| | |
|
| | |
|
| |
|
|
| |
This reverts commit b5c386ad7708ca9fe14f219b0ffc94cf6ce8fc88.
|
| | |
|
| | |
|
| |
|
|
|
|
|
| |
We used some shared SegmentVectors, e.g. for drawing raster or background tiles.
In longer running maps, this lead to resource accumulation. By storing the SegmentVectors
and the contained gfx::DrawScope objects, we ensure that resources get released
when the associated render objects vanish.
|
| | |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Before this change, the `GeoJSONVTData` instance was retained at the scheduled
lambda, which run on the worker thread represented by the `GeoJSONVTData::scheduler`
class member:
```
std::weak_ptr<GeoJSONVTData> weak = shared_from_this();
scheduler->scheduleAndReplyValue(
[id, weak, this]() -> TileFeatures {
if (auto self = weak.lock()) {
return impl.getTile(id.z, id.x, id.y).features;
}
return {};
},
fn);
```
It caused program termination in case `self` turned to be the last reference to `this`,
as the `std::thread` destructor was called from the thread it represented.
Now, only the `GeoJSONVTData::impl` class member is retained.
|
| |
|
|
|
|
|
|
|
|
| |
* [core] Refactoring transform state class
* [core] update matrix in transform, Fix precision
* [core] Make matrix mutable so that we only update them when needed
* [core] Add getters for matrices
|
| | |
|
| | |
|
| |
|
|
|
|
|
|
|
|
|
| |
This commit refactors `utils::ThreadPool` into a template
`ThreadedScheduler` class and provides aux type aliases.
So that it is possible to obtain a sequenced schedule,where
all the scheduled tasks are guarantied to be executed
consequently.
The sequenced lightweight scheduler is required by both the
orchestration thread and the refactored `FileSource` implementation.
|
| |
|
|
| |
So that it is possible to schedule normal `std::function` and use `mapbox::base::WeakPtr`.
|
| | |
|
| | |
|
| |
|
|
| |
It depends on platform specific compression library.
|
| |
|
|
|
|
|
| |
When compiling with gcc 8.3 under linux, I've got compiler error caused
by the following warning:
../src/mbgl/util/chrono.cpp:26:26: error: ‘__builtin___snprintf_chk’ output may be truncated before the last format character [-Werror=format-truncation=]
I need to slightly increase the buffer size to get rid of the warning.
|
| |
|
|
|
|
|
| |
Remove unecessary tile coordinate unproject -> project step.
Unproject+project isn't always a noop and this was the issue leading to wrong cover calculation in high pitch values.
Fixes: https://github.com/mapbox/mapbox-gl-native/issues/15442
|
| |
|
|
| |
This enables proper round-tripping for serialized format expression.
|
| |
|
|
|
|
|
|
|
|
| |
This change forces glyphs whose natural orientation in vertical writing
mode is 'sideways' to be rendered in upright orientation (only for non complex
text layouts). This is different compared to W3C / browser behavior that is by
default, renders glyphs in their respective natural orientation.
In the future, there might need to add a new layout property that would control
glyph orientation separately (e.g., text-glyph-orientation: natural | upright).
|
| | |
|
| | |
|
| |
|
|
| |
Fixes #14993
|
| | |
|
| |
|
|
| |
creating/destroying core threads
|
| | |
|
| |
|
|
|
|
|
|
|
|
| |
- Do not carry it over everywhere as parameter, it is a shared
instance anyway and the lifecycle is pretty much the app lifecycle
from the moment we instantiate a map.
- Rename to BackgroundScheduler because it is a Scheduler that will
do tasks in the background, we don't make assumptions if it is a
thread pool or a single thread.
- Most importantly, remove the dependency from `core` on `platform`.
|
| | |
|
| |
|
|
| |
Updated script detection files to reflect changes in Unicode 12.1.
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
