Refactor CreateAndRenderBitmap in Renderer class: change return type to void, improve error handling for palette setting
This commit is contained in:
@@ -56,14 +56,14 @@ class Renderer {
|
|||||||
bitmap->UpdateTexture(renderer_.get());
|
bitmap->UpdateTexture(renderer_.get());
|
||||||
}
|
}
|
||||||
|
|
||||||
absl::Status CreateAndRenderBitmap(int width, int height, int depth,
|
void CreateAndRenderBitmap(int width, int height, int depth,
|
||||||
const std::vector<uint8_t> &data,
|
const std::vector<uint8_t> &data,
|
||||||
gfx::Bitmap &bitmap,
|
gfx::Bitmap &bitmap, gfx::SnesPalette &palette) {
|
||||||
gfx::SnesPalette &palette) {
|
|
||||||
bitmap.Create(width, height, depth, data);
|
bitmap.Create(width, height, depth, data);
|
||||||
RETURN_IF_ERROR(bitmap.SetPalette(palette));
|
if (!bitmap.SetPalette(palette).ok()) {
|
||||||
|
throw std::runtime_error("Failed to set palette");
|
||||||
|
}
|
||||||
RenderBitmap(&bitmap);
|
RenderBitmap(&bitmap);
|
||||||
return absl::OkStatus();
|
|
||||||
}
|
}
|
||||||
|
|
||||||
private:
|
private:
|
||||||
|
|||||||
Reference in New Issue
Block a user