You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@httpd.apache.org by GitBox <gi...@apache.org> on 2021/02/19 07:46:28 UTC
[GitHub] [httpd] PooiaFerdowsi opened a new pull request #171: Update apreq.h
PooiaFerdowsi opened a new pull request #171:
URL: https://github.com/apache/httpd/pull/171
multiply the CPU usage
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@httpd.apache.org
For additional commands, e-mail: notifications-help@httpd.apache.org
[GitHub] [httpd] PooiaFerdowsi commented on pull request #171: Update apreq.h
Posted by GitBox <gi...@apache.org>.
PooiaFerdowsi commented on pull request #171:
URL: https://github.com/apache/httpd/pull/171#issuecomment-782127568
no
code optimization .
read :
https://stackoverflow.com/questions/6879114/multiplication-optimization
On Fri, Feb 19, 2021 at 4:48 PM Eric Covener <no...@github.com>
wrote:
> Is there some environment where this affects what happens at runtime?
> https://en.wikipedia.org/wiki/Constant_folding
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/apache/httpd/pull/171#issuecomment-782068579>, or
> unsubscribe
> <https://github.com/notifications/unsubscribe-auth/ASHDNERSULHE7DAD6O2WE2LS7ZQJXANCNFSM4X34OWHA>
> .
>
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@httpd.apache.org
For additional commands, e-mail: notifications-help@httpd.apache.org
[GitHub] [httpd] covener commented on pull request #171: Update apreq.h
Posted by GitBox <gi...@apache.org>.
covener commented on pull request #171:
URL: https://github.com/apache/httpd/pull/171#issuecomment-782042318
Why? It seems more readable beforehand.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@httpd.apache.org
For additional commands, e-mail: notifications-help@httpd.apache.org
[GitHub] [httpd] rpluem commented on pull request #171: Update apreq.h
Posted by GitBox <gi...@apache.org>.
rpluem commented on pull request #171:
URL: https://github.com/apache/httpd/pull/171#issuecomment-783342729
gcc does these precalculations.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@httpd.apache.org
For additional commands, e-mail: notifications-help@httpd.apache.org
[GitHub] [httpd] covener removed a comment on pull request #171: Update apreq.h
Posted by GitBox <gi...@apache.org>.
covener removed a comment on pull request #171:
URL: https://github.com/apache/httpd/pull/171#issuecomment-782042318
Why? It seems more readable beforehand.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@httpd.apache.org
For additional commands, e-mail: notifications-help@httpd.apache.org
[GitHub] [httpd] rpluem commented on pull request #171: Update apreq.h
Posted by GitBox <gi...@apache.org>.
rpluem commented on pull request #171:
URL: https://github.com/apache/httpd/pull/171#issuecomment-783223953
I don't see a benefit here as modern compilers precalculate "static" calculations like this during compilation and in the end the values you calculated manually will be used anyway during runtime. Hence we can use the more readable ones.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@httpd.apache.org
For additional commands, e-mail: notifications-help@httpd.apache.org
[GitHub] [httpd] PooiaFerdowsi closed pull request #171: Update apreq.h
Posted by GitBox <gi...@apache.org>.
PooiaFerdowsi closed pull request #171:
URL: https://github.com/apache/httpd/pull/171
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@httpd.apache.org
For additional commands, e-mail: notifications-help@httpd.apache.org
[GitHub] [httpd] PooiaFerdowsi commented on pull request #171: Update apreq.h
Posted by GitBox <gi...@apache.org>.
PooiaFerdowsi commented on pull request #171:
URL: https://github.com/apache/httpd/pull/171#issuecomment-782057967
HI Eric .
This is for the CPU .
multiply and plus.
int x = 5+5; // C
if than faster of
int x = 2*5; // C
and this is for program usage and not for readability .
Thanks .
On Fri, Feb 19, 2021 at 3:52 PM Eric Covener <no...@github.com>
wrote:
> Why? It seems more readable beforehand.
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/apache/httpd/pull/171#issuecomment-782042318>, or
> unsubscribe
> <https://github.com/notifications/unsubscribe-auth/ASHDNETCNDIK336RECL755DS7ZJZTANCNFSM4X34OWHA>
> .
>
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@httpd.apache.org
For additional commands, e-mail: notifications-help@httpd.apache.org
[GitHub] [httpd] PooiaFerdowsi commented on pull request #171: Update apreq.h
Posted by GitBox <gi...@apache.org>.
PooiaFerdowsi commented on pull request #171:
URL: https://github.com/apache/httpd/pull/171#issuecomment-783282145
yes .
I don't know .
My compiler is gcc .
What is Apache httpd ( Apache http server ) compiler ?
gcc support this technology ( I don't know )
Okay .
Thanks
On Mon, Feb 22, 2021 at 12:46 PM Ruediger Pluem <no...@github.com>
wrote:
> I don't see a benefit here as modern compilers precalculate "static"
> calculations like this during compilation and in the end the values you
> calculated manually will be used anyway during runtime. Hence we can use
> the more readable ones.
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/apache/httpd/pull/171#issuecomment-783223953>, or
> unsubscribe
> <https://github.com/notifications/unsubscribe-auth/ASHDNEQP54HX566RAHG7DEDTAIOIDANCNFSM4X34OWHA>
> .
>
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@httpd.apache.org
For additional commands, e-mail: notifications-help@httpd.apache.org
[GitHub] [httpd] covener commented on pull request #171: Update apreq.h
Posted by GitBox <gi...@apache.org>.
covener commented on pull request #171:
URL: https://github.com/apache/httpd/pull/171#issuecomment-782068579
Is there some environment where this affects what happens at runtime? https://en.wikipedia.org/wiki/Constant_folding
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@httpd.apache.org
For additional commands, e-mail: notifications-help@httpd.apache.org