Memory Issues about php7cc HOT 9 CLOSED

@andreimavenhut @vingiarrusso Can you please check out the code from the memory-issues branch and try again? It should require considerably less memory. Although I have to say that 35 MB still may not be enough if your projects are sufficiently large. If this is the case, the only advice I can give is to increase PHP memory limit.

from php7cc.

Thank you, I'm having this problem as well.

from php7cc.

@sstalle It worked much better on that branch. I still hit the memory limit but it went significantly farther than before. Will look into your suggestion. Fantastic tool you've created, thanks again!

from php7cc.

@vingiarrusso Is the project you are trying to check public? Can you provide a link to it so that I can try to investigate further?

from php7cc.

Confirmed. We see this too. PHP Fatal error: Allowed memory size of 536870912 bytes exhausted (tried to allocate 134217728 bytes) in phar:///php7cc.phar/src/Token/TokenCollection.php on line 18

from php7cc.

@seansan Did you try the code from the current master branch?

from php7cc.

@sstalle tried current master branch, error stills triggering.
Installed by composer.

memory_limit 256M

from php7cc.

easy fix for me was just to manually set a higher memory limit in the executable file (php7cc/bin/php7cc). Worked great running in the root folder of my fairly large project after that.

from php7cc.

Some more improvements reducing memory usage have been made. 90% of memory is consumed by PHP-Parser on an average run now, and I don't think that its memory usage can be brought down significantly. I think that current memory handling is efficient enough, as checking 14024 files requires 69.2 MB of memory, as reported by Blackfire:

Checked 14024 files in 1397.730 seconds

Blackfire Run completed
 Profile URL:  https://blackfire.io/profiles/...
  Total time:  23 min 18 s
    CPU time:  22 min 48 s
         I/O:       30.2 s
      Memory:      69.2 MB
     Network:          n/a

I've also added a note about increasing memory limit to readme in case someone needs that.

from php7cc.