Update code as per Drupal Coding standard - includes/context-task-handler.inc

Hi Madhav,
There are trailing spaces in the patch. Can you pleae look into this.
line numbers are :
20
61
71
96
99
116
129
140
173
189
216

Removed Whitespaces.

Some good changes in here, but a couple of issues with the patch in #4:

1. +++ b/includes/context-task-handler.inc
   @@ -16,24 +16,22 @@
   - * Render a context type task handler given a list of handlers
   - * attached to a type.
   + * Render a context type task handler.
     *
   - * @param $task
   + * It will give a list of handlers attached to a type.
   

   This change has (unintentionally?) changed the meaning here. I don't think this change should be included. We just need to re-wrap the comment to 80-chars.

2. +++ b/includes/context-task-handler.inc
   @@ -222,7 +223,7 @@ function ctools_context_handler_render_handler($task, $subtask, $handler, $conte
   - * Default function to provide contextual link for a task as defined by the handler.
   + * To provide contextual link for a task as defined by the handler.
   

   We've lost the mention of 'default function' here.

3. +++ b/includes/context-task-handler.inc
   @@ -259,17 +262,19 @@ function ctools_task_handler_default_contextual_link($handler, $plugin, $context
     *   TRUE if these contexts match the selection rules. NULL or FALSE
     *   otherwise.
   

   This could be re-wrapped to a single line that fits within 80 chars.

4. +++ b/includes/context-task-handler.inc
   @@ -288,11 +293,11 @@ function ctools_context_handler_select($handler, $contexts) {
   - * @param $handler
   + * @param object $handler
     *   The handler to be checked.
   

   @return tag is missing.

5. +++ b/includes/context-task-handler.inc
   @@ -364,11 +369,12 @@ function ctools_context_handler_get_all_contexts($task, $subtask, $handler) {
   - * Create an object suitable for use with the context system that kind of
   - * expects things in a certain, kind of clunky format.
   + * Create an object suitable for use with the context system.
   + *
   + * That kind of object expects things in a certain, kind of clunky format.
   

   This re-writing doesn't make much sense.

Unable to apply patch. Patch needs to be reroll.

Coding standard issues are fixed as per said in #6.

Can we have the interdiff please?

Adding interdiff.

Line exceeds 80 characters; contains 84 characters.
Doc comment short description must be on a single line, further text should be a separate paragraph.

Revised and did more coding standards

Solved

Hi guys.

I applied the patch and #14 makes sense for me.

RTBC

Thank you very much @tiago.urbano

Regards

With the release of the latest stable, most of this was picked off with phpcbf before the release.

Not sure there will get anything out of a reroll, closing this one.