From ce2adfaf4ad36174579ed36a7e31fffed42240a9 Mon Sep 17 00:00:00 2001 From: Christian Flothmann Date: Sun, 23 Nov 2014 20:13:47 +0100 Subject: [PATCH] removed unneeded check It is already checked in the `isEnabled()` method of the parent `ServerCommand` class if the current PHP build is HHVM and the `server:start` command is never enabled then. Thus, it's not needed to check for HHVM on every command execution. This was pointed out by @stof in #12489 for the `server:run` command. --- .../Bundle/FrameworkBundle/Command/ServerStartCommand.php | 5 ----- 1 file changed, 5 deletions(-) diff --git a/src/Symfony/Bundle/FrameworkBundle/Command/ServerStartCommand.php b/src/Symfony/Bundle/FrameworkBundle/Command/ServerStartCommand.php index c5ec95d4d4..f5fe6b6921 100644 --- a/src/Symfony/Bundle/FrameworkBundle/Command/ServerStartCommand.php +++ b/src/Symfony/Bundle/FrameworkBundle/Command/ServerStartCommand.php @@ -71,11 +71,6 @@ EOF */ protected function execute(InputInterface $input, OutputInterface $output) { - if (defined('HHVM_VERSION')) { - $output->writeln('This command is not supported on HHVM.'); - - return 1; - } if (!extension_loaded('pcntl')) { $output->writeln('This command needs the pcntl extension to run.'); $output->writeln('You can either install it or use the server:run command instead to run the built-in web server.');