Remove unused local variable and add return value to docs in _search_find_match_with_simplify() of 'search.module'.

This is correct. That local variable is indeed never used, and the return docs are correct. Should be rc eligible because it is just removing one line of cruft and adding API docs.

Fixing issue title though, since the patch is just adding the missing return type to the function docs, not fixing some incorrect docs.

Patch #5 is not quite right. It is missing the |null from the other patch in the return line... and the other patch I just noticed should be |null not |NULL in the @return line.

Changed NULL to null

Thanks! All correct now. Adding RC note to issue summary.

On of the Test is aborted, its happening with other patches also. Could you suggest how to handle this.

Recreated the new patch with exactly same changes.

The test is passed now, Keeping back to RTBC

Thanks! Hm. there used to be a "retest" button.... I guess that doesn't exist in the current test environment.

Anyway you probably could have just set the status back to RTBC and the test would have eventually been re-run. Next time. :)

Someone keeps aborting tests on the bot and that is causing issues to get set to Needs Work. We need to fix that. Anyway this is still RTBC.

Committed/pushed to 8.0.x, thanks!