[PATCH 4/6] dwrite: Use consistent trace format for rendering parameters object.
Nikolay Sivov
nsivov at codeweavers.com
Wed Dec 2 05:44:04 CST 2020
Signed-off-by: Nikolay Sivov <nsivov at codeweavers.com>
---
dlls/dwrite/main.c | 140 +++++++++++++++++++++++++--------------------
1 file changed, 78 insertions(+), 62 deletions(-)
diff --git a/dlls/dwrite/main.c b/dlls/dwrite/main.c
index cfb2e9f6a56..dd9bd4dc6a3 100644
--- a/dlls/dwrite/main.c
+++ b/dlls/dwrite/main.c
@@ -54,14 +54,15 @@ BOOL WINAPI DllMain(HINSTANCE hinstDLL, DWORD reason, LPVOID reserved)
return TRUE;
}
-struct renderingparams {
+struct renderingparams
+{
IDWriteRenderingParams3 IDWriteRenderingParams3_iface;
- LONG ref;
+ LONG refcount;
- FLOAT gamma;
- FLOAT contrast;
- FLOAT grayscalecontrast;
- FLOAT cleartype_level;
+ float gamma;
+ float contrast;
+ float grayscalecontrast;
+ float cleartype_level;
DWRITE_PIXEL_GEOMETRY geometry;
DWRITE_RENDERING_MODE1 mode;
DWRITE_GRID_FIT_MODE gridfit;
@@ -74,9 +75,7 @@ static inline struct renderingparams *impl_from_IDWriteRenderingParams3(IDWriteR
static HRESULT WINAPI renderingparams_QueryInterface(IDWriteRenderingParams3 *iface, REFIID riid, void **obj)
{
- struct renderingparams *This = impl_from_IDWriteRenderingParams3(iface);
-
- TRACE("(%p)->(%s %p)\n", This, debugstr_guid(riid), obj);
+ TRACE("%p, %s, %p.\n", iface, debugstr_guid(riid), obj);
if (IsEqualIID(riid, &IID_IDWriteRenderingParams3) ||
IsEqualIID(riid, &IID_IDWriteRenderingParams2) ||
@@ -96,51 +95,61 @@ static HRESULT WINAPI renderingparams_QueryInterface(IDWriteRenderingParams3 *if
static ULONG WINAPI renderingparams_AddRef(IDWriteRenderingParams3 *iface)
{
- struct renderingparams *This = impl_from_IDWriteRenderingParams3(iface);
- ULONG ref = InterlockedIncrement(&This->ref);
- TRACE("(%p)->(%d)\n", This, ref);
- return ref;
+ struct renderingparams *params = impl_from_IDWriteRenderingParams3(iface);
+ ULONG refcount = InterlockedIncrement(¶ms->refcount);
+
+ TRACE("%p, refcount %d.\n", iface, refcount);
+
+ return refcount;
}
static ULONG WINAPI renderingparams_Release(IDWriteRenderingParams3 *iface)
{
- struct renderingparams *This = impl_from_IDWriteRenderingParams3(iface);
- ULONG ref = InterlockedDecrement(&This->ref);
+ struct renderingparams *params = impl_from_IDWriteRenderingParams3(iface);
+ ULONG refcount = InterlockedDecrement(¶ms->refcount);
- TRACE("(%p)->(%d)\n", This, ref);
+ TRACE("%p, refcount %d.\n", iface, refcount);
- if (!ref)
- heap_free(This);
+ if (!refcount)
+ heap_free(params);
- return ref;
+ return refcount;
}
-static FLOAT WINAPI renderingparams_GetGamma(IDWriteRenderingParams3 *iface)
+static float WINAPI renderingparams_GetGamma(IDWriteRenderingParams3 *iface)
{
- struct renderingparams *This = impl_from_IDWriteRenderingParams3(iface);
- TRACE("(%p)\n", This);
- return This->gamma;
+ struct renderingparams *params = impl_from_IDWriteRenderingParams3(iface);
+
+ TRACE("%p.\n", iface);
+
+ return params->gamma;
}
-static FLOAT WINAPI renderingparams_GetEnhancedContrast(IDWriteRenderingParams3 *iface)
+static float WINAPI renderingparams_GetEnhancedContrast(IDWriteRenderingParams3 *iface)
{
- struct renderingparams *This = impl_from_IDWriteRenderingParams3(iface);
- TRACE("(%p)\n", This);
- return This->contrast;
+ struct renderingparams *params = impl_from_IDWriteRenderingParams3(iface);
+
+ TRACE("%p.\n", iface);
+
+ return params->contrast;
}
-static FLOAT WINAPI renderingparams_GetClearTypeLevel(IDWriteRenderingParams3 *iface)
+static float WINAPI renderingparams_GetClearTypeLevel(IDWriteRenderingParams3 *iface)
{
- struct renderingparams *This = impl_from_IDWriteRenderingParams3(iface);
- TRACE("(%p)\n", This);
- return This->cleartype_level;
+ struct renderingparams *params = impl_from_IDWriteRenderingParams3(iface);
+
+ TRACE("%p.\n", iface);
+
+ return params->cleartype_level;
}
static DWRITE_PIXEL_GEOMETRY WINAPI renderingparams_GetPixelGeometry(IDWriteRenderingParams3 *iface)
{
- struct renderingparams *This = impl_from_IDWriteRenderingParams3(iface);
- TRACE("(%p)\n", This);
- return This->geometry;
+ struct renderingparams *params = impl_from_IDWriteRenderingParams3(iface);
+
+ TRACE("%p.\n", iface);
+
+ return params->geometry;
}
static DWRITE_RENDERING_MODE rendering_mode_from_mode1(DWRITE_RENDERING_MODE1 mode)
@@ -161,35 +170,42 @@ static DWRITE_RENDERING_MODE rendering_mode_from_mode1(DWRITE_RENDERING_MODE1 mo
static DWRITE_RENDERING_MODE WINAPI renderingparams_GetRenderingMode(IDWriteRenderingParams3 *iface)
{
- struct renderingparams *This = impl_from_IDWriteRenderingParams3(iface);
+ struct renderingparams *params = impl_from_IDWriteRenderingParams3(iface);
- TRACE("(%p)\n", This);
+ TRACE("%p.\n", iface);
- return rendering_mode_from_mode1(This->mode);
+ return rendering_mode_from_mode1(params->mode);
}
-static FLOAT WINAPI renderingparams1_GetGrayscaleEnhancedContrast(IDWriteRenderingParams3 *iface)
+static float WINAPI renderingparams1_GetGrayscaleEnhancedContrast(IDWriteRenderingParams3 *iface)
{
- struct renderingparams *This = impl_from_IDWriteRenderingParams3(iface);
- TRACE("(%p)\n", This);
- return This->grayscalecontrast;
+ struct renderingparams *params = impl_from_IDWriteRenderingParams3(iface);
+
+ TRACE("%p.\n", iface);
+
+ return params->grayscalecontrast;
}
static DWRITE_GRID_FIT_MODE WINAPI renderingparams2_GetGridFitMode(IDWriteRenderingParams3 *iface)
{
- struct renderingparams *This = impl_from_IDWriteRenderingParams3(iface);
- TRACE("(%p)\n", This);
- return This->gridfit;
+ struct renderingparams *params = impl_from_IDWriteRenderingParams3(iface);
+
+ TRACE("%p.\n", iface);
+
+ return params->gridfit;
}
static DWRITE_RENDERING_MODE1 WINAPI renderingparams3_GetRenderingMode1(IDWriteRenderingParams3 *iface)
{
- struct renderingparams *This = impl_from_IDWriteRenderingParams3(iface);
- TRACE("(%p)\n", This);
- return This->mode;
+ struct renderingparams *params = impl_from_IDWriteRenderingParams3(iface);
+
+ TRACE("%p.\n", iface);
+
+ return params->mode;
}
-static const struct IDWriteRenderingParams3Vtbl renderingparamsvtbl = {
+static const struct IDWriteRenderingParams3Vtbl renderingparamsvtbl =
+{
renderingparams_QueryInterface,
renderingparams_AddRef,
renderingparams_Release,
@@ -203,11 +219,11 @@ static const struct IDWriteRenderingParams3Vtbl renderingparamsvtbl = {
renderingparams3_GetRenderingMode1
};
-static HRESULT create_renderingparams(FLOAT gamma, FLOAT contrast, FLOAT grayscalecontrast, FLOAT cleartype_level,
+static HRESULT create_renderingparams(float gamma, float contrast, float grayscalecontrast, float cleartype_level,
DWRITE_PIXEL_GEOMETRY geometry, DWRITE_RENDERING_MODE1 mode, DWRITE_GRID_FIT_MODE gridfit,
IDWriteRenderingParams3 **params)
{
- struct renderingparams *This;
+ struct renderingparams *object;
*params = NULL;
@@ -217,21 +233,21 @@ static HRESULT create_renderingparams(FLOAT gamma, FLOAT contrast, FLOAT graysca
if ((UINT32)gridfit > DWRITE_GRID_FIT_MODE_ENABLED || (UINT32)geometry > DWRITE_PIXEL_GEOMETRY_BGR)
return E_INVALIDARG;
- This = heap_alloc(sizeof(struct renderingparams));
- if (!This) return E_OUTOFMEMORY;
+ if (!(object = heap_alloc(sizeof(*object))))
+ return E_OUTOFMEMORY;
- This->IDWriteRenderingParams3_iface.lpVtbl = &renderingparamsvtbl;
- This->ref = 1;
+ object->IDWriteRenderingParams3_iface.lpVtbl = &renderingparamsvtbl;
+ object->refcount = 1;
- This->gamma = gamma;
- This->contrast = contrast;
- This->grayscalecontrast = grayscalecontrast;
- This->cleartype_level = cleartype_level;
- This->geometry = geometry;
- This->mode = mode;
- This->gridfit = gridfit;
+ object->gamma = gamma;
+ object->contrast = contrast;
+ object->grayscalecontrast = grayscalecontrast;
+ object->cleartype_level = cleartype_level;
+ object->geometry = geometry;
+ object->mode = mode;
+ object->gridfit = gridfit;
- *params = &This->IDWriteRenderingParams3_iface;
+ *params = &object->IDWriteRenderingParams3_iface;
return S_OK;
}
--
2.29.2
More information about the wine-devel
mailing list