Clean up API docs for includes directory, files starting with D-G

Cleanup for character limit, lines above @return, and @see/@ingroup positioning. No fixes to summaries or file docblocks yet. Waiting on #1315992: No standard for documenting menu callbacks and perhaps #1317826: Provide general documentation standard for callbacks.

Untagging.

This patch is still marked needs work, meaning it's not finished. Also, this and the other two through Z are actually going to be discarded and re-done, so don't waste time on 'em. :)

Postponing for core directory change.

Going to roll this in steps with interdiffs so that it's easier to review.

First, just moving the @see to the end.

Adding blank lines above @return.

Rewrapping lines over 80 chars + \n for non-summaries.

Updated issue summary.

Updated issue summary.

Cleaning up @file blocks. Next comes the fun part.

Summary corrections through errors.inc.

Updated issue summary.

Updated issue summary.

Updated issue summary.

Updated issue summary.

Summary corrections through file.inc.

I also removed a floating docblock with nothing attached to it--at first it looked like it might be a defgroup, but it's actually in the middle of another defgroup. So I added that to the followup list.

This patch isn't done yet.

@aspilicious and I checked and his comment in #14 is actually not accurate, at least in the current standard. See, for example: http://drupal.org/node/1354#forms So leaving those for now.

Still two more files to go. Hopefully the interdiffs help, although this one is turning out to be a lot smaller than I feared.

Through form.inc. Not nearly as bad as I thought.

I added a return value to one function here because it was already provided in the multi-line summary that I fixed.

And, done.

Theoretically one can just read the interdiffs for a first pass at a review, or to do a partial review. Or, if preferred, one can just read the final patch, which actually didn't turn out to be so terribly big.

Stripping a "Paths: " line per #1315992: No standard for documenting menu callbacks.

+++ b/core/includes/file.incundefined
@@ -1950,7 +1946,7 @@ function file_transfer($uri, $headers) {
 /**
- * Menu handler for private file transfers.
+ * Page callback: Handles private file transfers.
  *
  * Call modules that implement hook_file_download() to find out if a file is
  * accessible and what headers it should be transferred with. If one or more
@@ -1960,6 +1956,7 @@ function file_transfer($uri, $headers) {

@@ -1960,6 +1956,7 @@ function file_transfer($uri, $headers) {
  * If the file does not exist drupal_not_found() will be returned.
  *
  * @see hook_file_download()
+ * @see system_menu()
  */

+++ b/core/includes/form.incundefined
@@ -2901,7 +2921,9 @@ function date_validate($element) {
 /**
- * Helper function for usage with drupal_map_assoc to display month names.
+ * Renders a month name for display.
+ *
+ * Callback for drupal_map_assoc() within form_process_date().
  */

Looks like these are the only changes that need to be removed from the backport.

I don't see this commit in D8 yet.

Backport looks keen.

Whoops. Sorry @ksenzee.

Updated issue summary.

Updated issue summary.