From 1450c0e8d6adb3607a8271ceb958f8737f9a916f Mon Sep 17 00:00:00 2001 From: Joshua Spence Date: Fri, 25 Jul 2014 15:59:17 +1000 Subject: [PATCH] Omit unnecessary function call Summary: As mentioned on rP8ce35e6b67e7e2a81b274bab7a6dd19dedb4df06, `setConcreteOnly(true)` can be omitted since (lacking magical powers) `loadObjects()` will always only instantiate concrete objects. Test Plan: N/A Reviewers: epriestley, #blessed_reviewers Reviewed By: epriestley, #blessed_reviewers Subscribers: epriestley, Korvin Differential Revision: https://secure.phabricator.com/D10050 --- src/applications/conduit/method/ConduitQueryConduitAPIMethod.php | 1 - 1 file changed, 1 deletion(-) diff --git a/src/applications/conduit/method/ConduitQueryConduitAPIMethod.php b/src/applications/conduit/method/ConduitQueryConduitAPIMethod.php index bd97c02682..b30112e7c9 100644 --- a/src/applications/conduit/method/ConduitQueryConduitAPIMethod.php +++ b/src/applications/conduit/method/ConduitQueryConduitAPIMethod.php @@ -26,7 +26,6 @@ final class ConduitQueryConduitAPIMethod extends ConduitAPIMethod { $classes = id(new PhutilSymbolLoader()) ->setAncestorClass('ConduitAPIMethod') ->setType('class') - ->setConcreteOnly(true) ->loadObjects(); $names_to_params = array();