Skip to content

W
wine-winehq
Commit feca77fb authored by Nikolay Sivov's avatar Nikolay Sivov Committed by Alexandre Julliard
Browse files
Options

dwrite: Set isNewline cluster flag.

Signed-off-by: 's avatarNikolay Sivov <nsivov@codeweavers.com> Signed-off-by: 's avatarAlexandre Julliard <julliard@winehq.org>
parent 8b831b9d
Hide whitespace changes
Inline Side-by-side
Showing with 22 additions and 14 deletions
dlls/dwrite/analyzer.c
View file @ feca77fb
......@@ -326,6 +326,12 @@ static inline void set_break_condition(UINT32 pos, enum BreakConditionLocation l
}
}
BOOL lb_is_newline_char(WCHAR ch)
{
short c = get_table_entry(wine_linebreak_table, ch);
return c == b_LF || c == b_NL || c == b_CR || c == b_BK;
}
static HRESULT analyze_linebreaks(const WCHAR *text, UINT32 count, DWRITE_LINE_BREAKPOINT *breakpoints)
{
struct linebreaking_state state;
......
dlls/dwrite/dwrite_private.h
View file @ feca77fb
......@@ -139,6 +139,7 @@ extern BOOL is_face_type_supported(DWRITE_FONT_FACE_TYPE) DECLSPEC_HIDDEN;
extern HRESULT get_family_names_from_stream(IDWriteFontFileStream*,UINT32,DWRITE_FONT_FACE_TYPE,IDWriteLocalizedStrings**) DECLSPEC_HIDDEN;
extern HRESULT create_colorglyphenum(FLOAT,FLOAT,const DWRITE_GLYPH_RUN*,const DWRITE_GLYPH_RUN_DESCRIPTION*,DWRITE_MEASURING_MODE,
const DWRITE_MATRIX*,UINT32,IDWriteColorGlyphRunEnumerator**) DECLSPEC_HIDDEN;
extern BOOL lb_is_newline_char(WCHAR) DECLSPEC_HIDDEN;
/* Opentype font table functions */
struct dwrite_font_props {
......
dlls/dwrite/layout.c
View file @ feca77fb
......@@ -613,7 +613,7 @@ static inline void init_cluster_metrics(const struct dwrite_textlayout *layout,
if (metrics->length == 1) {
DWRITE_LINE_BREAKPOINT bp = get_effective_breakpoint(layout, position);
metrics->isWhitespace = bp.isWhitespace;
metrics->isNewline = FALSE /* FIXME */;
metrics->isNewline = metrics->canWrapLineAfter && lb_is_newline_char(layout->str[position]);
metrics->isSoftHyphen = bp.isSoftHyphen;
}
else {
......
dlls/dwrite/tests/layout.c
View file @ feca77fb
......@@ -1700,13 +1700,13 @@ static void test_GetClusterMetrics(void)
/* SP - SPACE */ 0x20
};
static const WCHAR str5W[] = {'a','\r','b','\n','c','\n','\r','d','\r','\n','e',0xb,'f',0xc,
'g',0x0085,'h',0x2028,'i',0x2029,0xad,0};
'g',0x0085,'h',0x2028,'i',0x2029,0xad,0xa,0};
static const WCHAR str3W[] = {0x2066,')',')',0x661,'(',0x627,')',0};
static const WCHAR str2W[] = {0x202a,0x202c,'a',0};
static const WCHAR strW[] = {'a','b','c','d',0};
static const WCHAR str4W[] = {'a',' ',0};
DWRITE_INLINE_OBJECT_METRICS inline_metrics;
DWRITE_CLUSTER_METRICS metrics[21];
DWRITE_CLUSTER_METRICS metrics[22];
IDWriteTextLayout1 *layout1;
IDWriteInlineObject *trimm;
IDWriteTextFormat *format;
......@@ -1997,9 +1997,8 @@ todo_wine
memset(metrics, 0, sizeof(metrics));
hr = IDWriteTextLayout_GetClusterMetrics(layout, metrics, sizeof(metrics)/sizeof(metrics[0]), &count);
ok(hr == S_OK, "got 0x%08x\n", hr);
ok(count == 21, "got %u\n", count);
ok(count == 22, "got %u\n", count);
todo_wine {
ok(metrics[1].isNewline == 1, "got %d\n", metrics[1].isNewline);
ok(metrics[3].isNewline == 1, "got %d\n", metrics[3].isNewline);
ok(metrics[5].isNewline == 1, "got %d\n", metrics[5].isNewline);
......@@ -2010,7 +2009,8 @@ todo_wine {
ok(metrics[15].isNewline == 1, "got %d\n", metrics[15].isNewline);
ok(metrics[17].isNewline == 1, "got %d\n", metrics[17].isNewline);
ok(metrics[19].isNewline == 1, "got %d\n", metrics[19].isNewline);
}
ok(metrics[21].isNewline == 1, "got %d\n", metrics[21].isNewline);
ok(metrics[0].isNewline == 0, "got %d\n", metrics[0].isNewline);
ok(metrics[2].isNewline == 0, "got %d\n", metrics[2].isNewline);
ok(metrics[4].isNewline == 0, "got %d\n", metrics[4].isNewline);
......@@ -2021,12 +2021,16 @@ todo_wine {
ok(metrics[14].isNewline == 0, "got %d\n", metrics[14].isNewline);
ok(metrics[16].isNewline == 0, "got %d\n", metrics[16].isNewline);
ok(metrics[18].isNewline == 0, "got %d\n", metrics[18].isNewline);
ok(metrics[20].isNewline == 0, "got %d\n", metrics[20].isNewline);
for (i = 0; i < count; i++) {
ok(metrics[i].length == 1, "%d: got %d\n", i, metrics[i].length);
ok(metrics[i].isSoftHyphen == (i == count - 1), "%d: got %d\n", i, metrics[i].isSoftHyphen);
ok(metrics[i].isSoftHyphen == (i == count - 2), "%d: got %d\n", i, metrics[i].isSoftHyphen);
if (metrics[i].isNewline) {
ok(metrics[i].width == 0.0f, "%u: got width %f\n", i, metrics[i].width);
if (i == 17 || i == 19)
todo_wine ok(metrics[i].width == 0.0f, "%u: got width %f\n", i, metrics[i].width);
else
ok(metrics[i].width == 0.0f, "%u: got width %f\n", i, metrics[i].width);
ok(metrics[i].canWrapLineAfter == 1, "%u: got %d\n", i, metrics[i].canWrapLineAfter);
}
}
......@@ -3341,12 +3345,11 @@ static void test_GetLineMetrics(void)
count = 2;
hr = IDWriteTextLayout_GetLineMetrics(layout, metrics, 2, &count);
ok(hr == S_OK, "got 0x%08x\n", hr);
todo_wine {
ok(count == 2, "got %u\n", count);
/* baseline is relative to a line, and is not accumulated */
ok(metrics[0].baseline == metrics[1].baseline, "got %.2f, %.2f\n", metrics[0].baseline,
metrics[1].baseline);
}
IDWriteTextLayout_Release(layout);
IDWriteTextFormat_Release(format);
......@@ -3364,20 +3367,19 @@ todo_wine {
todo_wine
ok(count == 6, "got %u\n", count);
todo_wine {
ok(metrics[0].length == 2, "got %u\n", metrics[0].length);
ok(metrics[1].length == 2, "got %u\n", metrics[1].length);
ok(metrics[2].length == 2, "got %u\n", metrics[2].length);
ok(metrics[3].length == 1, "got %u\n", metrics[3].length);
ok(metrics[4].length == 3, "got %u\n", metrics[4].length);
todo_wine
ok(metrics[5].length == 0, "got %u\n", metrics[5].length);
}
todo_wine {
ok(metrics[0].newlineLength == 1, "got %u\n", metrics[0].newlineLength);
ok(metrics[1].newlineLength == 1, "got %u\n", metrics[1].newlineLength);
ok(metrics[2].newlineLength == 1, "got %u\n", metrics[2].newlineLength);
ok(metrics[3].newlineLength == 1, "got %u\n", metrics[3].newlineLength);
todo_wine {
ok(metrics[4].newlineLength == 2, "got %u\n", metrics[4].newlineLength);
ok(metrics[5].newlineLength == 0, "got %u\n", metrics[5].newlineLength);
}
......@@ -4008,7 +4010,6 @@ static void test_SetWordWrapping(void)
count = 0;
hr = IDWriteTextLayout_GetLineMetrics(layout, NULL, 0, &count);
ok(hr == E_NOT_SUFFICIENT_BUFFER, "got 0x%08x\n", hr);
todo_wine
ok(count == 2, "got %u\n", count);
hr = IDWriteTextLayout_SetWordWrapping(layout, DWRITE_WORD_WRAPPING_WRAP);
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment