Namespace docs standard fix in ConfigFactory.php

There were only two changes that I saw.

Given what is defined in #1 in the documentation

When using a class or interface name immediately after any @-tag in documentation, prefix it with the fully-qualified namespace name (starting with a backslash).

I've re-added all the starting backslashes to the namespaces.

This patch contains all changes from the first patch as well.

+++ b/core/lib/Drupal/Core/Config/ConfigFactory.phpundefined
@@ -15,36 +15,35 @@
+   * @param \Symfony\Component\EventDispatcher\EventDispatcher
    *   An event dispatcher instance to use for configuration events.

This is missing the variable name $event_dispatcher. This was an already existing error. that should be corrected in this line change.

@Lars Toomre Thanks for catching that. I've corrected that line here.

You are welcome @aaronott. The one final change that I do not think belongs in this patch is removing the blank line before the closing brace of a class declaration.

There is some uncertainty about what the Drupal standard is for class declarations and where blank lines are required. However, @tim.plunkett, @xjm and @sun have all stated that a blank line before the closing brace is part of our standards. Hence, I would leave out the removal of that blank line at the end of the patch in #4.

Interesting I'll keep that in mind.

I've removed the removal of that blank line from patch #4.

@YesCT The interdiff looks like the following

diff --git a/core/lib/Drupal/Core/Config/ConfigFactory.php b/core/lib/Drupal/Core/Config/ConfigFactory.php
index 7cab5cf..e3c40bd 100644
--- a/core/lib/Drupal/Core/Config/ConfigFactory.php
+++ b/core/lib/Drupal/Core/Config/ConfigFactory.php
@@ -84,4 +84,5 @@ public function get($name) {
     $config = new Config($name, $this->storage, $this->eventDispatcher);
     return $config->init();
   }
+
 }

Let me know if there is anything else. This is just adding back a blank line that I initially removed.

This is my first day reviewing patches for Drupal and getting started (again) with contributing to Drupal here at DrupalCampDelhi - http://drupalcampdelhi.com

* This simple issue helped me learn about Doxygen comment formatting conventions for Drupal - http://drupal.org/node/1354#namespaces

* It also taught me about interdiffs - http://drupal.org/node/1488712

The patch in #6 addresses the issue. The latest patch in #10 is not the real patch but the interdiff, so I am going ahead and marking this RTBC. Thanks @aaronott and @YesCT :-)

I remember changing the status :p

This patch looks great to me, but is likely to collide with #1648930: Introduce configuration schema and use for translation which is tagged as "Avoid commit conflicts." So we can commit this soon, just not right now. :)

The standard for using namespaces in docs is currently under question, so I would advise not committing this or working further on it until
#1487760: [policy, no patch] Decide on documentation standards for namespaced items
is figured out. Sigh.

Updated issue summary to make it clear this is a separate issue and not a conflicting follow up

That's obsolete now I think:
https://www.drupal.org/node/1487760#comment-9138221