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

PremultipliedImage/UnassociatedImage: Add support for different backing storage #9333

Open
kkaefer opened this issue Jun 21, 2017 · 7 comments
Labels
Core The cross-platform C++ core, aka mbgl performance Speed, stability, CPU usage, memory usage, or power usage

Comments

@kkaefer
Copy link
Member

kkaefer commented Jun 21, 2017

On Android, we're currently copying over the pixels from the underlying Android Bitmap when we initialize an Image. Instead, we should change our Image implementation on Android so that it always uses java Bitmap object as the backing store to avoid converting back and forth.

/cc @asheemmamoowala

@kkaefer kkaefer added Core The cross-platform C++ core, aka mbgl performance Speed, stability, CPU usage, memory usage, or power usage labels Jun 21, 2017
@asheemmamoowala
Copy link
Contributor

Android P introduces a replacement for BitmapFactory that may help with this: ImageDecoder

@kkaefer
Copy link
Member Author

kkaefer commented Mar 8, 2018

Great, that means we can switch over once 2024 rolls around :)

@stale
Copy link

stale bot commented Oct 27, 2018

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the archived Archived because of inactivity label Oct 27, 2018
@stale
Copy link

stale bot commented Dec 6, 2018

This issue has been automatically detected as stale because it has not had recent activity and will be archived. Thank you for your contributions.

@stale stale bot closed this as completed Dec 6, 2018
@kkaefer kkaefer reopened this Jan 2, 2019
@stale stale bot removed the archived Archived because of inactivity label Jan 2, 2019
@stale stale bot added the archived Archived because of inactivity label Jul 1, 2019
@stale
Copy link

stale bot commented Jul 2, 2019

This issue has been automatically detected as stale because it has not had recent activity and will be archived. Thank you for your contributions.

@stale stale bot closed this as completed Jul 2, 2019
@tobrun tobrun reopened this Jul 2, 2019
@stale stale bot removed the archived Archived because of inactivity label Jul 2, 2019
@stale stale bot added the archived Archived because of inactivity label Dec 29, 2019
@stale
Copy link

stale bot commented Dec 29, 2019

This issue has been automatically detected as stale because it has not had recent activity and will be archived. Thank you for your contributions.

@stale stale bot closed this as completed Dec 29, 2019
@stale stale bot removed the archived Archived because of inactivity label Jan 2, 2020
@stale stale bot added the archived Archived because of inactivity label Jul 1, 2020
@stale
Copy link

stale bot commented Jul 3, 2020

This issue has been automatically detected as stale because it has not had recent activity and will be archived. Thank you for your contributions.

@stale stale bot closed this as completed Jul 3, 2020
@springmeyer springmeyer reopened this Jul 3, 2020
@stale stale bot removed the archived Archived because of inactivity label Jul 3, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Core The cross-platform C++ core, aka mbgl performance Speed, stability, CPU usage, memory usage, or power usage
Projects
None yet
Development

No branches or pull requests

4 participants