You are viewing a plain text version of this content. The canonical link for it is here.
Posted to bugs@httpd.apache.org by bu...@apache.org on 2011/06/07 04:11:58 UTC

DO NOT REPLY [Bug 51330] New: Code clean up (merge ap_rputs calls in mod_status.c)

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

             Bug #: 51330
           Summary: Code clean up (merge ap_rputs calls in mod_status.c)
           Product: Apache httpd-2
           Version: 2.3-HEAD
          Platform: PC
        OS/Version: Windows 2000
            Status: NEW
          Severity: trivial
          Priority: P2
         Component: mod_status
        AssignedTo: bugs@httpd.apache.org
        ReportedBy: christophe.jaillet@wanadoo.fr
    Classification: Unclassified


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

Hi,

here is a patch that merges some adjacent calls to 'ap_rputs'.

-- 
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@httpd.apache.org
For additional commands, e-mail: bugs-help@httpd.apache.org


DO NOT REPLY [Bug 51330] Code clean up (merge ap_rputs calls in mod_status.c)

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=51330

Stefan Fritsch <sf...@sfritsch.de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|REOPENED                    |RESOLVED
         Resolution|                            |FIXED

--- Comment #4 from Stefan Fritsch <sf...@sfritsch.de> 2011-06-13 11:13:14 UTC ---
committed as r1135085

-- 
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@httpd.apache.org
For additional commands, e-mail: bugs-help@httpd.apache.org


DO NOT REPLY [Bug 51330] Code clean up (merge ap_rputs calls in mod_status.c)

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=51330

Christophe JAILLET <ch...@wanadoo.fr> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |PatchAvailable

-- 
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@httpd.apache.org
For additional commands, e-mail: bugs-help@httpd.apache.org


DO NOT REPLY [Bug 51330] Code clean up (merge ap_rputs calls in mod_status.c)

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=51330

Christophe JAILLET <ch...@wanadoo.fr> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |REOPENED
         Resolution|FIXED                       |

--- Comment #3 from Christophe JAILLET <ch...@wanadoo.fr> 2011-06-09 13:32:41 UTC ---
In fact, r1131465 was about inlining 'ap_rputs'.

The attached patch is for merging several calls to 'ap_rputs' into one.


Both ideas (inlining and merging calls when appropriate) were described here :
http://mail-archives.apache.org/mod_mbox/httpd-dev/201105.mbox/%3Cis0sae$i4k$1@dough.gmane.org%3E

For me this patch is definitively something different than r1131465.

-- 
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@httpd.apache.org
For additional commands, e-mail: bugs-help@httpd.apache.org


DO NOT REPLY [Bug 51330] Code clean up (merge ap_rputs calls in mod_status.c)

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=51330

Igor Galić <i....@brainsware.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED

--- Comment #2 from Igor Galić <i....@brainsware.org> 2011-06-09 00:22:46 UTC ---
I just realized this has been filed against trunk, so I'm closing the issue.

-- 
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@httpd.apache.org
For additional commands, e-mail: bugs-help@httpd.apache.org


DO NOT REPLY [Bug 51330] Code clean up (merge ap_rputs calls in mod_status.c)

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=51330

Igor Galić <i....@brainsware.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |FixedInTrunk

--- Comment #1 from Igor Galić <i....@brainsware.org> 2011-06-09 00:21:53 UTC ---
r1131465

-- 
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@httpd.apache.org
For additional commands, e-mail: bugs-help@httpd.apache.org


DO NOT REPLY [Bug 51330] Code clean up (merge ap_rputs calls in mod_status.c)

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=51330

Christophe JAILLET <ch...@wanadoo.fr> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|FixedInTrunk                |

-- 
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@httpd.apache.org
For additional commands, e-mail: bugs-help@httpd.apache.org