Mailing-List: contact bugs-help@apr.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@apr.apache.org
From: bugzilla@apache.org
To: bugs@apr.apache.org
Subject: DO NOT REPLY [Bug 52350] New: Code clean up (apr_strings.c)
Date: Sat, 17 Dec 2011 07:14:29 +0000
Message-ID: <bug-52350-43213@https.issues.apache.org/bugzilla/>
Auto-Submitted: auto-generated
Content-Type: text/plain; charset="UTF-8"
MIME-Version: 1.0

https://issues.apache.org/bugzilla/show_bug.cgi?id=52350

             Bug #: 52350
           Summary: Code clean up (apr_strings.c)
           Product: APR
           Version: HEAD
          Platform: PC
        OS/Version: Windows 2000
            Status: NEW
          Severity: enhancement
          Priority: P2
         Component: APR
        AssignedTo: bugs@apr.apache.org
        ReportedBy: christophe.jaillet@wanadoo.fr
    Classification: Unclassified


Created attachment 28081
  --> https://issues.apache.org/bugzilla/attachment.cgi?id=28081
Proposed patch

Hi,

in strings/apr_strings.c, most functions allocate its own memory when needed
except 'apr_pstrdup' which actually call 'apr_pmemdup' for that.

This patch inlines this call in order to be more consistent with-in the code.
This also removes a small calling overhead in case the compiler does not inline
the function by itself.

Hoping this helps.

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: bugs-unsubscribe@apr.apache.org
For additional commands, e-mail: bugs-help@apr.apache.org