logo       

Re: Corrected bug in StringHelper::endsWith on CVS HEAD: msg#00026

Subject: Re: Corrected bug in StringHelper::endsWith on CVS HEAD

On Mar 9, 2005, at 6:01 AM, Björn Carlsson wrote:

It is checked the row above this row, I quote: "if (suffix.length() <= s.length()) {"

And I should be a bit more clear on anopther point to. This is my suggestion for solution, I have not commited this code, since I'm not a member of the project. And I post it here since I'm not a member of log4cxx-dev@xxxxxxxxxxxxxxxxxxx either. If I find more things I want to correct, I will consider to find some other way.

 /Björn Carlsson
VersionSupport.com


Thanks for both catches. The best way to report bugs and submit patches is to use the log4cxx JIRA (http://nagoya.apache.org/jira). I've opened and resolved a Jira bug report (LOGCXX-70) and committed unit tests for startsWith (which had a different bug) and endsWith and the corrections.

I'll use the still open LOGCXX-67 bug that added the NTEventLogTestCase to resolve the other issue.

I've emailed Apache infrastructure about the mailing list archive problems.




<Prev in Thread] Current Thread [Next in Thread>
Google Custom Search

Recently Viewed:
science.linguis...    culture.sf.lite...    video.mplayer.c...    yellowdog.gener...    ietf.rfc822/199...    emacs.help/2002...    redhat.release....    kernel.speakup/...    java.openejb.de...    debian.devel.gt...    xfree86.newbie/...    bug-tracking.ma...    pam/2003-05/msg...    games.devel.ope...    user-groups.lin...    music.pancham/2...    network.mq.deve...    web.html.genera...    arklinux.bugs/2...    linux.ecasound/...    qnx.openqnx.dev...    org.user-groups...    file-systems.sf...    trustix.contrib...   
Home | blog view | USPTO Patent Archive | advertise | OSDir is an inevitable website. super tiny logo

Free Magazines

Cisco News
Receive a free quarterly e-newsletter with exclusive articles on how Cisco IT uses its own products and solutions to enable the business.
subscribe

Systems Management News, the newspaper for IT systems administration and data center managers! Each issue of Systems Management News is chock-full of news and analysis to help you understand what's happening in your field.
subscribe

The Enterprise Newsweekly eWeek is the essential technology information source for builders of e-business.
subscribe

Oracle Magazine Oracle Magazine contains technology strategy articles, sample code, tips, Oracle and partner news, how to articles for developers and DBAs, and more. Oracle (NASDAQ: ORCL) is the world's largest enterprise software company.
subscribe

Total Telecom Total Telecom is "The Economist of the communications industry".
subscribe