On Thu, Apr 10, 2008 at 5:00 AM, sum1abi@gmail.com <sum1abi@gmail.com> wrote:
>
> On Wed, Apr 9, 2008 at 4:26 PM, Dominic Lachowicz
> <domlachowicz@gmail.com> wrote:
> >
> > Hi Xun,
> >
> > + if (m_sBackupLatex.size())
> > + m_sBackupLatex.clear();
> >
> > Please just call clear().
>
> +void AP_Dialog_Latex::_backupLatex(void)
> +{
>
> + if (m_sBackupLatex.size())
> + m_sBackupLatex.clear();
> + m_sBackupLatex += m_sLatex;
> +}
> +
>
> It's not clear to me why you need to use the += operator. I would
> just remove the clear() call and use 'm_sBackupLatex = m_sLatex;' in
> that function.
>
Yes I should. Clearly I missed the fact that they already point to
different strings when we need to compare them. Thanks for reviewing.
Attached is a revised(-again) patch.
-- Thanks & regards Xun Sun
This archive was generated by hypermail 2.1.8 : Thu Apr 10 2008 - 04:38:14 CEST