<div dir="ltr"><br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">
OUString RectanglePointToOUString(const css::drawing::RectanglePoint& rp){<br>
    if(rp == css::drawing::RectanglePoint::<wbr>LEFT_TOP)<br>
        return OUString::createFromAscii("<wbr>LEFT_TOP");<br>...    <br>
    SAL_WARN("sd","Unknown RectanglePoint Value");<br>
    return OUString::createFromAscii("");<br>
}<br>
<br></blockquote><div><br></div><div>The existing practice is to use a switch statement, not a sequence of if statements. Also, as css::drawing::RectanglePoint is a new-style so-called scoped enum (enum class, not plain enum), you should introduce a stream output operator<< directly, no need for a ToString function just for SAL_INFO etc usage. If you have an operator<< for it, you can use values of that css::drawing::RectanglePoint type directly in SAL_INFO etc.</div><div><br></div><div>See for instance include/sfx2/event.hxx, this:</div><div><br></div><div><p style="margin:0px;font-stretch:normal;font-size:11px;line-height:normal;font-family:Menlo;color:rgb(0,0,0)"><span style="font-variant-ligatures:no-common-ligatures">template< typename charT, typename traits ></span></p>
<p style="margin:0px;font-stretch:normal;font-size:11px;line-height:normal;font-family:Menlo;color:rgb(0,0,0)"><span style="font-variant-ligatures:no-common-ligatures">inline std::basic_ostream<charT, traits> & </span><span style="font-variant-ligatures:no-common-ligatures;color:rgb(255,255,255);background-color:rgb(0,0,0)">operator</span><span style="font-variant-ligatures:no-common-ligatures"> <<(</span></p>
<p style="margin:0px;font-stretch:normal;font-size:11px;line-height:normal;font-family:Menlo;color:rgb(0,0,0)"><span style="font-variant-ligatures:no-common-ligatures">    std::basic_ostream<charT, traits> & stream, const SvMacroItemId& id )</span></p>
<p style="margin:0px;font-stretch:normal;font-size:11px;line-height:normal;font-family:Menlo;color:rgb(0,0,0)"><span style="font-variant-ligatures:no-common-ligatures">{</span></p>
<p style="margin:0px;font-stretch:normal;font-size:11px;line-height:normal;font-family:Menlo;color:rgb(0,0,0)"><span style="font-variant-ligatures:no-common-ligatures">    switch(id)</span></p>
<p style="margin:0px;font-stretch:normal;font-size:11px;line-height:normal;font-family:Menlo;color:rgb(0,0,0)"><span style="font-variant-ligatures:no-common-ligatures">    {</span></p></div><div><span style="font-variant-ligatures:no-common-ligatures">...</span></div><div><span style="font-variant-ligatures:no-common-ligatures"><br></span></div><div><span style="font-variant-ligatures:no-common-ligatures">That is my code, and my style, and it is I who (as far as I have noticed) have been adding such debug output operators when I have had the need. Nobody has opposed, so that is the existing style you should follow. If somebody now opposes, they are free to introduce another style, *if* they change there existing ones to be of that style, too. Just my opinion, of course.</span></div><div><span style="font-variant-ligatures:no-common-ligatures"><br></span></div><div><span style="font-variant-ligatures:no-common-ligatures">Note that instead of a SAL_WARN as you have, I output the undefined value as a number. That is more useful.</span></div><div><br></div><div>--tml</div></div></div></div>