apigen / ApiGen

src/Console/Environment/MemoryLimit.php

<?php

/**
 * This file is part of the ApiGen (http://apigen.org)
 *
 * For the full copyright and license information, please view
 * the file LICENSE that was distributed with this source code.
 */

namespace ApiGen\Console\Environment;

class MemoryLimit
{

    /**
     * @param string $newMemoryLimit
     */
    public function setMemoryLimitTo($newMemoryLimit)
    {
        if (function_exists('ini_set')) {
            $memoryLimit = trim(ini_get('memory_limit'));
            if ($memoryLimit !== -1 && $this->getMemoryInBytes($memoryLimit) < 512 * 1024 * 1024) {

The strict comparison !== seems to always evaluate to true as the types of $memoryLimit (string) and -1 (integer) can never be identical. Maybe you want to use a loose comparison != instead?

                @ini_set('memory_limit', $newMemoryLimit);
            }
            unset($memoryInBytes, $memoryLimit);
        }
    }


    /**
     * @param string $value
     * @return int
     */
    private function getMemoryInBytes($value)
    {
        $unit = strtolower(substr($value, -1, 1));
        $value = (int) $value;
        if ($unit === 'g') {
            return $value * 1024 * 1024 * 1024;
        }
        if ($unit === 'm') {
            return $value * 1024 * 1024;
        }
        if ($unit === 'k') {
            return $value * 1024;
        }
    }
}