Set hard-coded defaults for list<string> values to array()

Summary: These should default to array() so they're safe to `foreach` over.

Test Plan: Grepped for 'list<string>'.

Reviewers: codeblock, btrahan, starruler, vrana

Reviewed By: vrana

CC: aran

Differential Revision: https://secure.phabricator.com/D4600

src/applications/config/option/PhabricatorCoreConfigOptions.php

@@ -72,7 +72,7 @@ final class PhabricatorCoreConfigOptions
             "and a call to 'Leap Into Action'. If you'd prefer more ".
             "traditional UI strings like 'Submit', you can set this flag to ".
             "disable most of the jokes and easter eggs.")),
-       $this->newOption('environment.append-paths', 'list<string>', null)
+       $this->newOption('environment.append-paths', 'list<string>', array())
         ->setSummary(
           pht("These paths get appended to your \$PATH envrionment variable."))
         ->setDescription(

src/applications/config/option/PhabricatorExtendingPhabricatorConfigOptions.php

@@ -13,11 +13,11 @@ final class PhabricatorExtendingPhabricatorConfigOptions
 
   public function getOptions() {
     return array(
-      $this->newOption('load-libraries', 'list<string>', null)
+      $this->newOption('load-libraries', 'list<string>', array())
         ->setLocked(true)
         ->setSummary(pht("Paths to additional phutil libraries to load."))
         ->addExample('/srv/our-libs/sekrit-phutil', pht('Valid Setting')),
-      $this->newOption('events.listeners', 'list<string>', null)
+      $this->newOption('events.listeners', 'list<string>', array())
         ->setLocked(true)
         ->setSummary(
           pht("Listeners receive callbacks when interesting things occur."))

src/applications/differential/config/PhabricatorDifferentialConfigOptions.php

@@ -22,7 +22,7 @@ final class PhabricatorDifferentialConfigOptions
       $this->newOption(
         'differential.custom-remarkup-rules',
         'list<string>',
-        null)
+        array())
         ->setSummary(pht('Custom remarkup rules.'))
         ->setDescription(
           pht(
@@ -31,7 +31,7 @@ final class PhabricatorDifferentialConfigOptions
       $this->newOption(
         'differential.custom-remarkup-block-rules',
         'list<string>',
-        null)
+        array())
         ->setSummary(pht('Custom remarkup block rules.'))
         ->setDescription(
           pht(
@@ -146,7 +146,7 @@ final class PhabricatorDifferentialConfigOptions
             "revision. If this option is false, Differential defaults to the ".
             "best it can, something like 'George Washington' or ".
             "'gwashington'.")),
-      $this->newOption('differential.generated-paths', 'list<string>', null)
+      $this->newOption('differential.generated-paths', 'list<string>', array())
         ->setSummary(pht("File regexps to treat as automatically generated."))
         ->setDescription(
           pht(