scansbr: Removed the ``performance hack'' for inc as it wasn't faster.
authormarkus schnalke <meillo@marmaro.de>
Thu, 22 Mar 2012 20:27:53 +0000 (21:27 +0100)
committermarkus schnalke <meillo@marmaro.de>
Thu, 22 Mar 2012 20:27:53 +0000 (21:27 +0100)
commitd7b4f0034bc4f5b5c2f990d0984858e9b6f4131a
treebe4d1e1138b96ea0986b524e12d3197b856bb077
parente51cee378fd6a0f8d6f3af2a125715036d1f2f8c
scansbr: Removed the ``performance hack'' for inc as it wasn't faster.
I tested a 150MB/500msgs mbox but couldn't see any time differences. In both
cases it took 2-5s on my machine. Hence, I go for the much simpler and not
``we use strange internals'' version. If you really inc much larger mboxes,
then just take some time. ;-)
The explaining comment for the hack had been:
    Performance hack: some people like to run "inc" on things like
    net.sources or large digests. We do a copy directly into the output
    buffer rather than going through an intermediate buffer.
        We need the amount of data m_getfld found & don't want to do a
    strlen on the long buffer so there's a hack in m_getfld to save the
    amount of data it returned in the global "msg_count".
uip/scansbr.c