Split core/include/fpdfapi/fpdf_resource.h

public/fpdf_edit.h

Index: public/fpdf_edit.h
diff --git a/public/fpdf_edit.h b/public/fpdf_edit.h
index 64eef263a551dc8443d78beb377fd8e9a4c34012..46d5b3d0886c11c7f425c02b91cb8ee925dfa7dc 100644
--- a/public/fpdf_edit.h
+++ b/public/fpdf_edit.h
@@ -16,7 +16,7 @@
 // or long integer numbers.
 
 #define FPDF_ARGB(a, r, g, b)                                     \
-  ((((uint32_t)(((uint8_t)(b) | ((FX_WORD)((uint8_t)(g)) << 8)) | \
+  ((((uint32_t)(((uint8_t)(b) | ((uint16_t)((uint8_t)(g)) << 8)) | \

[Tom Sepez, 2016/03/22 16:55:53]

Formatting, also this looks wrong.  We want to cast all of these to FX_DWORD or
uint32_t to avoid silent promotion to int for smaller types.

[dsinclair, 2016/03/23 00:00:25]

Done. You may want to verify my bracketing.

                 (((FX_DWORD)(uint8_t)(r)) << 16)))) |             \
    (((FX_DWORD)(uint8_t)(a)) << 24))
 #define FPDF_GetBValue(argb) ((uint8_t)(argb))