logging-log4cxx-user mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Tommi Maekitalo <to...@epgmbh.de>
Subject Re: bug in stringhelper
Date Fri, 14 Jan 2005 08:03:15 GMT
Hi,

sorry - I meant endsWith, which does not work. This was the fix for the bug I 
found.

I will look at Cppunit. I haven't used it before. And also at Jira. I don't 
know it either.


Tommi


Am Freitag, 14. Januar 2005 06:03 schrieb Curt Arnold:
> The patch you sent seems to be modifying StringHelper::endsWith, not
>
> ::startsWith and then intent was that
>
> StringHelper::endsWith("foobar", "bar") == true
>
> I believe the existing code would do that and the patched version would
> not but haven't had a chance to create a unit test case for those
> functions.
>
> Could you describe the symptom that you believe that a bug in
> StringHelper is causing? It would be helpful if you could also create a
> test case for every bug that you fix so that we can detect regressions.
> I'd also prefer that you use Jira to submit patches.
>
> On Jan 13, 2005, at 2:50 PM, Tommi Mäkitalo wrote:
> > Hi,
> >
> > There is a bug in StringHelper::startsWith:
> > "std::string::compare(size_type pos, size_type len, const std::string&
> > s)" compares a substring of the string with s and not vice versa.
> >
> > Here is a patch.
> >
> > After that it works at least with easy cases on my x86-64.
> >
> > Thank you Curt. I vote for a 0.9.8 soon.
> >
> >
> > Tommi
> > <stringhelper.diff>

Mime
View raw message