<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">2013/7/8 Miklos Vajna <span dir="ltr"><<a href="mailto:vmiklos@vmiklos.hu" target="_blank">vmiklos@vmiklos.hu</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Sun, Jul 07, 2013 at 11:19:11AM -0700, =?ISO-8859-1?Q?Zolnai_Tam=E1s_ <<a href="mailto:zolnaitamas2000@gmail.com">zolnaitamas2000@gmail.com</a>>, ?= wrote:<br>
> +    // character borders<br>
> +    editeng::SvxBorderLine*     m_pTopBorder;<br>
> +    editeng::SvxBorderLine*     m_pBottomBorder;<br>
> +    editeng::SvxBorderLine*     m_pRightBorder;<br>
> +    editeng::SvxBorderLine*     m_pLeftBorder;<br>
> +<br>
....<br>
> +void SwFont::SetLeftBorder( const editeng::SvxBorderLine* pLeftBorder )<br>
> +{<br>
> +    delete m_pLeftBorder;<br>
> +    if( pLeftBorder )<br>
> +        m_pLeftBorder = new editeng::SvxBorderLine(*pLeftBorder);<br>
> +    else<br>
> +        m_pLeftBorder = 0;<br>
> +    bFntChg = sal_True;<br>
> +}<br>
<br>
Hmm, what's the benefit or manual memory management here? If you need a<br>
pointer, you could use boost::shared_ptr here. Though I'm not sure you<br>
really need a pointer here. If you just want optional members,<br>
boost::optional is better.<br>
</blockquote></div><br></div><div class="gmail_extra">You're right, boost::optional will be better. I'll correct it.<br></div><div class="gmail_extra"><br></div></div>