<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, May 31, 2013 at 9:07 PM, Fabian Müntefering <span dir="ltr"><<a href="mailto:Cookiezzz97@gmx.de" target="_blank">Cookiezzz97@gmx.de</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Implemented functions of text format and text layout regarding text<br>
alignment.<br>
<br>
Try 3: Used proper enum value to set the enum variables.<br>
<br>
<br><br>
<br></blockquote></div><br><pre>+    if(alignment!=DWRITE_TEXT_ALIGNMENT_CENTER && alignment!=DWRITE_TEXT_ALIGNMENT_LEADING &&
+       alignment!=DWRITE_TEXT_ALIGNMENT_JUSTIFIED && alignment!=DWRITE_TEXT_ALIGNMENT_TRAILING)
+        return E_INVALIDARG;
+    This->format.text_align = alignment;<br><br></pre><pre>This doesn't look great, you'll need a test that shows it really rejects meaningless values,<br>and if it does it would be simpler to compare with greatest enum value.<br>
</pre></div></div>