From c27021764dba648b2501333805488e369d7eff22 Mon Sep 17 00:00:00 2001
From: Aurimas Fischer <aurimas@gmail.com>
Date: Sun, 1 Feb 2009 17:15:57 +0200
Subject: gdiplus: Implement GdipSetStringFormatFlags with tests

---
 dlls/gdiplus/stringformat.c       |    5 ++-
 dlls/gdiplus/tests/stringformat.c |   62 +++++++++++++++++++++++++++++++++++++
 2 files changed, 66 insertions(+), 1 deletions(-)

diff --git a/dlls/gdiplus/stringformat.c b/dlls/gdiplus/stringformat.c
index a28f014..21053b7 100644
--- a/dlls/gdiplus/stringformat.c
+++ b/dlls/gdiplus/stringformat.c
@@ -286,7 +286,10 @@ GpStatus WINGDIPAPI GdipSetStringFormatTrimming(GpStringFormat *format,
 
 GpStatus WINGDIPAPI GdipSetStringFormatFlags(GpStringFormat *format, INT flags)
 {
-    FIXME("format (%p) flags (%d)\n", format, flags);
+    if(!format)
+        return InvalidParameter;
+
+    format->attr = flags;
 
     return Ok;
 }
diff --git a/dlls/gdiplus/tests/stringformat.c b/dlls/gdiplus/tests/stringformat.c
index 9331912..35291ec 100644
--- a/dlls/gdiplus/tests/stringformat.c
+++ b/dlls/gdiplus/tests/stringformat.c
@@ -318,6 +318,67 @@ static void test_getgenericdefault(void)
     expect(Ok, stat);
 }
 
+static void test_stringformatflags(void)
+{
+    GpStringFormat *format;
+    GpStatus stat;
+
+    INT flags;
+
+    stat = GdipCreateStringFormat(0, LANG_NEUTRAL, &format);
+    expect(Ok, stat);
+
+    /* NULL args */
+    stat = GdipSetStringFormatFlags(NULL, NULL);
+    expect(InvalidParameter, stat);
+
+    stat = GdipSetStringFormatFlags(format, NULL);
+    expect(Ok, stat);
+    stat = GdipGetStringFormatFlags(format, &flags);
+    expect(Ok, stat);
+    expect(0, flags);
+
+    /* Check some valid flags */
+    stat = GdipSetStringFormatFlags(format, StringFormatFlagsDirectionRightToLeft);
+    expect(Ok, stat);
+    stat = GdipGetStringFormatFlags(format, &flags);
+    expect(Ok, stat);
+    expect(StringFormatFlagsDirectionRightToLeft, flags);
+
+    stat = GdipSetStringFormatFlags(format, StringFormatFlagsNoFontFallback);
+    expect(Ok, stat);
+    stat = GdipGetStringFormatFlags(format, &flags);
+    expect(Ok, stat);
+    expect(StringFormatFlagsNoFontFallback, flags);
+
+    /* Check some flag combinations */
+    stat = GdipSetStringFormatFlags(format, StringFormatFlagsDirectionVertical
+        | StringFormatFlagsNoFitBlackBox);
+    expect(Ok, stat);
+    stat = GdipGetStringFormatFlags(format, &flags);
+    expect(Ok, stat);
+    expect((StringFormatFlagsDirectionVertical
+        | StringFormatFlagsNoFitBlackBox), flags);
+
+    stat = GdipSetStringFormatFlags(format, StringFormatFlagsDisplayFormatControl
+        | StringFormatFlagsMeasureTrailingSpaces);
+    expect(Ok, stat);
+    stat = GdipGetStringFormatFlags(format, &flags);
+    expect(Ok, stat);
+    expect((StringFormatFlagsDisplayFormatControl
+        | StringFormatFlagsMeasureTrailingSpaces), flags);
+
+    /* Check invalid flags */
+    stat = GdipSetStringFormatFlags(format, 0xdeadbeef);
+    expect(Ok, stat);
+    stat = GdipGetStringFormatFlags(format, &flags);
+    expect(Ok, stat);
+    expect(0xdeadbeef, flags);
+
+    stat = GdipDeleteStringFormat(format);
+    expect(Ok, stat);
+}
+
 START_TEST(stringformat)
 {
     struct GdiplusStartupInput gdiplusStartupInput;
@@ -336,6 +397,7 @@ START_TEST(stringformat)
     test_getgenerictypographic();
     test_tabstops();
     test_getgenericdefault();
+    test_stringformatflags();
 
     GdiplusShutdown(gdiplusToken);
 }
-- 
1.6.0.4