Refactor bitmap update methods to remove unnecessary parameters; simplify texture updating in Renderer and Bitmap classes

src/app/core/platform/renderer.h

@@ -52,8 +52,8 @@ class Renderer {
   /**
    * @brief Used to update a bitmap on the screen.
    */
-  void UpdateBitmap(gfx::Bitmap *bitmap, bool use_sdl_update = false) {
+  void UpdateBitmap(gfx::Bitmap *bitmap) {
-    bitmap->UpdateTexture(renderer_.get(), use_sdl_update);
+    bitmap->UpdateTexture(renderer_.get());
   }
 
   absl::Status CreateAndRenderBitmap(int width, int height, int depth,

src/app/editor/dungeon/dungeon_editor.cc

@@ -119,14 +119,14 @@ absl::Status DungeonEditor::RefreshGraphics() {
     int block = rooms_[current_room_id_].blocks()[i];
     RETURN_IF_ERROR(graphics_bin_[block].ApplyPaletteWithTransparent(
         current_palette_group_[current_palette_id_], 0));
-    Renderer::GetInstance().UpdateBitmap(&graphics_bin_[block], true);
+    Renderer::GetInstance().UpdateBitmap(&graphics_bin_[block]);
   }
   auto sprites_aux1_pal_group = rom()->palette_group().sprites_aux1;
   for (int i = 9; i < 16; i++) {
     int block = rooms_[current_room_id_].blocks()[i];
     RETURN_IF_ERROR(graphics_bin_[block].ApplyPaletteWithTransparent(
         sprites_aux1_pal_group[current_palette_id_], 0));
-    Renderer::GetInstance().UpdateBitmap(&graphics_bin_[block], true);
+    Renderer::GetInstance().UpdateBitmap(&graphics_bin_[block]);
   }
   return absl::OkStatus();
 }

src/app/editor/graphics/graphics_editor.cc

@@ -294,7 +294,7 @@ absl::Status GraphicsEditor::UpdateGfxTabView() {
         auto draw_tile_event = [&]() {
           current_sheet_canvas_.DrawTileOnBitmap(tile_size_, &current_bitmap,
                                                  current_color_);
-          Renderer::GetInstance().UpdateBitmap(&current_bitmap, true);
+          Renderer::GetInstance().UpdateBitmap(&current_bitmap);
         };
 
         current_sheet_canvas_.UpdateColorPainter(
@@ -373,7 +373,7 @@ absl::Status GraphicsEditor::UpdatePaletteColumn() {
               ->data()[current_sheet_]
               .ApplyPaletteWithTransparent(palette, edit_palette_sub_index_));
       Renderer::GetInstance().UpdateBitmap(
-          &rom()->mutable_gfx_sheets()->data()[current_sheet_], true);
+          &rom()->mutable_gfx_sheets()->data()[current_sheet_]);
       refresh_graphics_ = false;
     }
   }

src/app/gfx/bitmap.cc

@@ -307,7 +307,7 @@ void Bitmap::CreateTexture(SDL_Renderer *renderer) {
   SDL_UnlockTexture(texture_.get());
 }
 
-void Bitmap::UpdateTexture(SDL_Renderer *renderer, bool use_sdl_update) {
+void Bitmap::UpdateTexture(SDL_Renderer *renderer) {
   SDL_Surface *converted_surface =
       SDL_ConvertSurfaceFormat(surface_.get(), SDL_PIXELFORMAT_ARGB8888, 0);
   if (converted_surface) {
@@ -319,28 +319,11 @@ void Bitmap::UpdateTexture(SDL_Renderer *renderer, bool use_sdl_update) {
 
   SDL_LockTexture(texture_.get(), nullptr, (void **)&texture_pixels,
                   &converted_surface_->pitch);
-  if (use_sdl_update) {
+  memcpy(texture_pixels, converted_surface_->pixels,
-    SDL_UpdateTexture(texture_.get(), nullptr, converted_surface_->pixels,
+         converted_surface_->h * converted_surface_->pitch);
-                      converted_surface_->pitch);
-  } else {
-    memcpy(texture_pixels, converted_surface_->pixels,
-           converted_surface_->h * converted_surface_->pitch);
-  }
   SDL_UnlockTexture(texture_.get());
 }
 
-void Bitmap::CreateTexture(std::shared_ptr<SDL_Renderer> renderer) {
-  texture_ = std::shared_ptr<SDL_Texture>{
-      SDL_CreateTextureFromSurface(renderer.get(), surface_.get()),
-      SDL_Texture_Deleter{}};
-}
-
-void Bitmap::UpdateTexture(std::shared_ptr<SDL_Renderer> renderer) {
-  texture_ = std::shared_ptr<SDL_Texture>{
-      SDL_CreateTextureFromSurface(renderer.get(), surface_.get()),
-      SDL_Texture_Deleter{}};
-}
-
 absl::Status Bitmap::ApplyPalette(const SnesPalette &palette) {
   if (surface_ == nullptr) {
     return absl::FailedPreconditionError(

src/app/gfx/bitmap.h

@@ -108,14 +108,12 @@ class Bitmap {
    * Converts the surface from a RGB to ARGB format.
    * Uses SDL_TEXTUREACCESS_STREAMING to allow for live updates.
    */
-  void CreateTexture(std::shared_ptr<SDL_Renderer> renderer);
+  void CreateTexture(SDL_Renderer *renderer);
 
   /**
    * @brief Updates the underlying SDL_Texture when it already exists.
    */
-  void UpdateTexture(std::shared_ptr<SDL_Renderer> renderer);
+  void UpdateTexture(SDL_Renderer *renderer);
-  void CreateTexture(SDL_Renderer *renderer);
-  void UpdateTexture(SDL_Renderer *renderer, bool use_sdl_update = false);
 
   /**
    * @brief Copy color data from the SnesPalette into the SDL_Palette