{"issue":{"state":"open","pull_request":{"diff_url":"https://github.com/fabpot/Silex/pull/205.diff","html_url":"https://github.com/fabpot/Silex/pull/205","patch_url":"https://github.com/fabpot/Silex/pull/205.patch"},"user":{"avatar_url":"https://secure.gravatar.com/avatar/55ae9bb3ed56705d67cdce4c7c4eff6f?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","gravatar_id":"55ae9bb3ed56705d67cdce4c7c4eff6f","url":"https://api.github.com/users/mgatto","login":"mgatto","id":341379},"milestone":null,"assignee":null,"labels":[],"updated_at":"2011-11-01T20:41:36Z","comments":2,"closed_at":null,"url":"https://api.github.com/repos/fabpot/Silex/issues/205","body":"Setting Application.php's constructor argument to true will cause Silex to use ApcUniversalClassLoader for a performance boost. Adding no argument to the constructor will cause Silex to use the default UniversalClassLoader as before.\r\n\r\nA constructor argument seems best since the class loader is initialized in the constructor, thus no $app['apc'] option could be used here.\r\n\r\nWill add tests if you think this is a reasonable change.","html_url":"https://github.com/fabpot/Silex/issues/205","number":205,"title":"Option to Use ApcUniversalClassLoader","id":2114708,"created_at":"2011-11-01T20:30:40Z"},"action":"created","comment":{"user":{"avatar_url":"https://secure.gravatar.com/avatar/55ae9bb3ed56705d67cdce4c7c4eff6f?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","gravatar_id":"55ae9bb3ed56705d67cdce4c7c4eff6f","url":"https://api.github.com/users/mgatto","login":"mgatto","id":341379},"updated_at":"2011-11-01T20:41:36Z","url":"https://api.github.com/repos/fabpot/Silex/issues/comments/2595626","body":"I would rather do something like this:\r\n\r\n```php\r\n<?php\r\n//possible new behavior; uses ApcUniversalClassLoader\r\n$app = new Application(array(\r\n    'apcClassLoader' => true,\r\n));\r\n```\r\n\r\nor even:\r\n\r\n```php\r\n<?php\r\n//possible new behavior; uses ApcUniversalClassLoader\r\n$app = new Application(array(\r\n    'apc.class.loader' => true,\r\n));\r\n```\r\n\r\ninstead of:\r\n\r\n```php\r\n<?php\r\n//possible new behavior; uses ApcUniversalClassLoader\r\n$app = new Application(true);\r\n```\r\n\r\nThoughts? Looks messy?\r\n\r\nBackward Compatible with current behavior\r\n-------------------------------------------------------------\r\n\r\n```php\r\n<?php\r\n//old behavior; uses UniversalClassLoader as before\r\n$app = new Application();\r\n```\r\n","id":2595626,"created_at":"2011-11-01T20:41:36Z"}}