summaryrefslogtreecommitdiff
path: root/Userland/Libraries/LibGfx
diff options
context:
space:
mode:
authorAndreas Kling <kling@serenityos.org>2021-03-16 12:00:43 +0100
committerAndreas Kling <kling@serenityos.org>2021-03-16 12:00:43 +0100
commit50233317268eaa6eb45b53820a30d4ece7847ae2 (patch)
tree75cbdb45cf7caafa386f53b9a288984e567d5f50 /Userland/Libraries/LibGfx
parente0f32626bcae634528327e43bcfa59c810c5a3ef (diff)
downloadserenity-50233317268eaa6eb45b53820a30d4ece7847ae2.zip
LibGfx: Rename 32-bit bitmap StorageFormats to BGRA8888 and BGRx8888
Diffstat (limited to 'Userland/Libraries/LibGfx')
-rw-r--r--Userland/Libraries/LibGfx/Bitmap.cpp4
-rw-r--r--Userland/Libraries/LibGfx/Bitmap.h32
2 files changed, 18 insertions, 18 deletions
diff --git a/Userland/Libraries/LibGfx/Bitmap.cpp b/Userland/Libraries/LibGfx/Bitmap.cpp
index 09e1505de5..5612fae972 100644
--- a/Userland/Libraries/LibGfx/Bitmap.cpp
+++ b/Userland/Libraries/LibGfx/Bitmap.cpp
@@ -64,8 +64,8 @@ size_t Bitmap::minimum_pitch(size_t physical_width, BitmapFormat format)
case StorageFormat::Indexed8:
element_size = 1;
break;
- case StorageFormat::RGB32:
- case StorageFormat::RGBA32:
+ case StorageFormat::BGRx8888:
+ case StorageFormat::BGRA8888:
element_size = 4;
break;
default:
diff --git a/Userland/Libraries/LibGfx/Bitmap.h b/Userland/Libraries/LibGfx/Bitmap.h
index 69204b405e..bb8c8e78cc 100644
--- a/Userland/Libraries/LibGfx/Bitmap.h
+++ b/Userland/Libraries/LibGfx/Bitmap.h
@@ -73,17 +73,17 @@ inline bool is_valid_bitmap_format(unsigned format)
enum class StorageFormat {
Indexed8,
- RGB32,
- RGBA32,
+ BGRx8888,
+ BGRA8888,
};
static StorageFormat determine_storage_format(BitmapFormat format)
{
switch (format) {
case BitmapFormat::BGRx8888:
- return StorageFormat::RGB32;
+ return StorageFormat::BGRx8888;
case BitmapFormat::BGRA8888:
- return StorageFormat::RGBA32;
+ return StorageFormat::BGRA8888;
case BitmapFormat::Indexed1:
case BitmapFormat::Indexed2:
case BitmapFormat::Indexed4:
@@ -295,14 +295,14 @@ inline const RGBA32* Bitmap::scanline(int y) const
}
template<>
-inline Color Bitmap::get_pixel<StorageFormat::RGB32>(int x, int y) const
+inline Color Bitmap::get_pixel<StorageFormat::BGRx8888>(int x, int y) const
{
VERIFY(x >= 0 && x < physical_width());
return Color::from_rgb(scanline(y)[x]);
}
template<>
-inline Color Bitmap::get_pixel<StorageFormat::RGBA32>(int x, int y) const
+inline Color Bitmap::get_pixel<StorageFormat::BGRA8888>(int x, int y) const
{
VERIFY(x >= 0 && x < physical_width());
return Color::from_rgba(scanline(y)[x]);
@@ -318,10 +318,10 @@ inline Color Bitmap::get_pixel<StorageFormat::Indexed8>(int x, int y) const
inline Color Bitmap::get_pixel(int x, int y) const
{
switch (determine_storage_format(m_format)) {
- case StorageFormat::RGB32:
- return get_pixel<StorageFormat::RGB32>(x, y);
- case StorageFormat::RGBA32:
- return get_pixel<StorageFormat::RGBA32>(x, y);
+ case StorageFormat::BGRx8888:
+ return get_pixel<StorageFormat::BGRx8888>(x, y);
+ case StorageFormat::BGRA8888:
+ return get_pixel<StorageFormat::BGRA8888>(x, y);
case StorageFormat::Indexed8:
return get_pixel<StorageFormat::Indexed8>(x, y);
default:
@@ -330,13 +330,13 @@ inline Color Bitmap::get_pixel(int x, int y) const
}
template<>
-inline void Bitmap::set_pixel<StorageFormat::RGB32>(int x, int y, Color color)
+inline void Bitmap::set_pixel<StorageFormat::BGRx8888>(int x, int y, Color color)
{
VERIFY(x >= 0 && x < physical_width());
scanline(y)[x] = color.value();
}
template<>
-inline void Bitmap::set_pixel<StorageFormat::RGBA32>(int x, int y, Color color)
+inline void Bitmap::set_pixel<StorageFormat::BGRA8888>(int x, int y, Color color)
{
VERIFY(x >= 0 && x < physical_width());
scanline(y)[x] = color.value(); // drop alpha
@@ -344,11 +344,11 @@ inline void Bitmap::set_pixel<StorageFormat::RGBA32>(int x, int y, Color color)
inline void Bitmap::set_pixel(int x, int y, Color color)
{
switch (determine_storage_format(m_format)) {
- case StorageFormat::RGB32:
- set_pixel<StorageFormat::RGB32>(x, y, color);
+ case StorageFormat::BGRx8888:
+ set_pixel<StorageFormat::BGRx8888>(x, y, color);
break;
- case StorageFormat::RGBA32:
- set_pixel<StorageFormat::RGBA32>(x, y, color);
+ case StorageFormat::BGRA8888:
+ set_pixel<StorageFormat::BGRA8888>(x, y, color);
break;
case StorageFormat::Indexed8:
VERIFY_NOT_REACHED();