From 765d1ebe41defa659a703a2f0ff5284c0da96a95 Mon Sep 17 00:00:00 2001 From: Lei Zhang Date: Wed, 21 Mar 2018 04:36:35 +0000 Subject: Rename one ArgbEncode() variant to AlphaAndColorRefToArgb(). Mark the other ArgbEncode() variant as constexpr. Change-Id: I84207d805766e5efc669638bd8a15cfead356a41 Reviewed-on: https://pdfium-review.googlesource.com/28577 Commit-Queue: Lei Zhang Reviewed-by: dsinclair --- core/fxge/dib/fx_dib_main.cpp | 6 +++--- core/fxge/fx_dib.h | 6 +++--- 2 files changed, 6 insertions(+), 6 deletions(-) (limited to 'core/fxge') diff --git a/core/fxge/dib/fx_dib_main.cpp b/core/fxge/dib/fx_dib_main.cpp index 5d7b585979..6c3a236edd 100644 --- a/core/fxge/dib/fx_dib_main.cpp +++ b/core/fxge/dib/fx_dib_main.cpp @@ -87,9 +87,9 @@ FX_COLORREF ArgbToColorRef(FX_ARGB argb) { return FXSYS_RGB(FXARGB_R(argb), FXARGB_G(argb), FXARGB_B(argb)); } -uint32_t ArgbEncode(int a, FX_COLORREF bgr) { - return FXARGB_MAKE(a, FXSYS_GetRValue(bgr), FXSYS_GetGValue(bgr), - FXSYS_GetBValue(bgr)); +FX_ARGB AlphaAndColorRefToArgb(int a, FX_COLORREF colorref) { + return FXARGB_MAKE(a, FXSYS_GetRValue(colorref), FXSYS_GetGValue(colorref), + FXSYS_GetBValue(colorref)); } FX_ARGB StringToFXARGB(const WideStringView& wsValue) { diff --git a/core/fxge/fx_dib.h b/core/fxge/fx_dib.h index 5dcfac4962..0878dfd7fd 100644 --- a/core/fxge/fx_dib.h +++ b/core/fxge/fx_dib.h @@ -99,11 +99,11 @@ std::pair ArgbToAlphaAndColorRef(FX_ARGB argb); // Returns FX_COLORREF. FX_COLORREF ArgbToColorRef(FX_ARGB argb); -inline FX_ARGB ArgbEncode(int a, int r, int g, int b) { +constexpr FX_ARGB ArgbEncode(int a, int r, int g, int b) { return (a << 24) | (r << 16) | (g << 8) | b; } -// TODO(thestig): Rename to AlphaAndColorRefToArgb(). -FX_ARGB ArgbEncode(int a, FX_COLORREF bgr); + +FX_ARGB AlphaAndColorRefToArgb(int a, FX_COLORREF colorref); FX_ARGB StringToFXARGB(const WideStringView& view); -- cgit v1.2.3