summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorTao Bao <tbao@google.com>2018-11-08 19:05:23 +0100
committerandroid-build-merger <android-build-merger@google.com>2018-11-08 19:05:23 +0100
commit0c295babc7041fd2ad1e162a9d7cac55e8747a0c (patch)
tree77873f71c56e2dd272c09f83bfa13f2f31e4ea58
parentMerge changes If24c6b7c,I381b0103 am: 67b7e64034 am: aeda63b106 (diff)
parentMerge "minui: GRSurface manages data with std::unique_ptr." am: 481613b35f (diff)
downloadandroid_bootable_recovery-0c295babc7041fd2ad1e162a9d7cac55e8747a0c.tar
android_bootable_recovery-0c295babc7041fd2ad1e162a9d7cac55e8747a0c.tar.gz
android_bootable_recovery-0c295babc7041fd2ad1e162a9d7cac55e8747a0c.tar.bz2
android_bootable_recovery-0c295babc7041fd2ad1e162a9d7cac55e8747a0c.tar.lz
android_bootable_recovery-0c295babc7041fd2ad1e162a9d7cac55e8747a0c.tar.xz
android_bootable_recovery-0c295babc7041fd2ad1e162a9d7cac55e8747a0c.tar.zst
android_bootable_recovery-0c295babc7041fd2ad1e162a9d7cac55e8747a0c.zip
-rw-r--r--minui/include/minui/minui.h14
-rw-r--r--minui/resources.cpp15
2 files changed, 16 insertions, 13 deletions
diff --git a/minui/include/minui/minui.h b/minui/include/minui/minui.h
index 0b499e621..3231248a0 100644
--- a/minui/include/minui/minui.h
+++ b/minui/include/minui/minui.h
@@ -17,6 +17,7 @@
#pragma once
#include <stdint.h>
+#include <stdlib.h>
#include <sys/types.h>
#include <functional>
@@ -32,7 +33,7 @@
class GRSurface {
public:
- virtual ~GRSurface();
+ virtual ~GRSurface() = default;
// Creates and returns a GRSurface instance that's sufficient for storing an image of the given
// size. The starting address of the surface data is aligned to SURFACE_DATA_ALIGNMENT. Returns
@@ -44,7 +45,7 @@ class GRSurface {
std::unique_ptr<GRSurface> Clone() const;
virtual uint8_t* data() {
- return data_;
+ return data_.get();
}
const uint8_t* data() const {
@@ -61,7 +62,14 @@ class GRSurface {
: width(width), height(height), row_bytes(row_bytes), pixel_bytes(pixel_bytes) {}
private:
- uint8_t* data_{ nullptr };
+ // The deleter for data_, whose data is allocated via aligned_alloc(3).
+ struct DataDeleter {
+ void operator()(uint8_t* data) {
+ free(data);
+ }
+ };
+
+ std::unique_ptr<uint8_t, DataDeleter> data_;
size_t data_size_;
DISALLOW_COPY_AND_ASSIGN(GRSurface);
diff --git a/minui/resources.cpp b/minui/resources.cpp
index 9027bc668..c7af1904d 100644
--- a/minui/resources.cpp
+++ b/minui/resources.cpp
@@ -46,24 +46,19 @@ std::unique_ptr<GRSurface> GRSurface::Create(int width, int height, int row_byte
auto result = std::unique_ptr<GRSurface>(new GRSurface(width, height, row_bytes, pixel_bytes));
result->data_size_ =
(data_size + kSurfaceDataAlignment - 1) / kSurfaceDataAlignment * kSurfaceDataAlignment;
- result->data_ = static_cast<uint8_t*>(aligned_alloc(kSurfaceDataAlignment, result->data_size_));
- if (result->data_ == nullptr) return nullptr;
+ result->data_.reset(
+ static_cast<uint8_t*>(aligned_alloc(kSurfaceDataAlignment, result->data_size_)));
+ if (!result->data_) return nullptr;
return result;
}
std::unique_ptr<GRSurface> GRSurface::Clone() const {
auto result = GRSurface::Create(width, height, row_bytes, pixel_bytes, data_size_);
- memcpy(result->data_, data_, data_size_);
+ if (!result) return nullptr;
+ memcpy(result->data(), data(), data_size_);
return result;
}
-GRSurface::~GRSurface() {
- if (data_ != nullptr) {
- free(data_);
- data_ = nullptr;
- }
-}
-
PngHandler::PngHandler(const std::string& name) {
std::string res_path = g_resource_dir + "/" + name + ".png";
png_fp_.reset(fopen(res_path.c_str(), "rbe"));