Code style cleanup of language system

Nice cleanup!

just noticed.

+++ b/core/lib/Drupal/Core/Language/LanguageDefault.php
@@ -1,5 +1,10 @@
+ * Contains \Drupal\Core\Language\LanguageDefault

Needs a period at the end.

Worth having a second look if there is anything else.

Thanks for the quick fix:) Back to RTBC.

Triaging the queue a bit.

Rerolled.

12: 2212411-code-style-cleanup-of-language-system-12.patch queued for re-testing.

Just a few remarks, otherwise looks good, thanks.

1. +++ b/core/lib/Drupal/Core/Language/LanguageDefault.php
   @@ -24,10 +29,11 @@ class LanguageDefault {
   +   * @param array $options
   ...
   +  public function __construct(array $options) {
   

   In the entity system we call the initial values param $values. I'd do the same for consistency.

2. +++ b/core/lib/Drupal/Core/Language/LanguageManagerInterface.php
   @@ -17,10 +17,10 @@
   -   * @param \use Drupal\Core\StringTranslation\TranslationInterface $translation
   +   * @param \Drupal\Core\StringTranslation\TranslationInterface $translation
   

   lol

3. +++ b/core/lib/Drupal/Core/Language/LanguageManagerInterface.php
   @@ -77,17 +77,17 @@ public function reset($type = NULL);
   +   *   (optional) Specifies the state of the languages that have to be
   +   *   returned. It can be: Language::STATE_CONFIGURABLE,
   

   Am I wrong or now "returned." fits in the upper line?

Patch from #12 applied. Tests cleanly.

Thanks @plach. Here's a reroll.

Found another comment that was running over 80 chars and a couple of typos, so here it is again.

Thanks!

Oh cool. This isn't nearly as big of a patch as I was fearing. :) I think it's cool to let this go in despite not being in a "disruptive patch period" since most of these fixes are pretty small and self-contained.

Committed and pushed to 8.x. Thanks!