Pass docstring into `docstringContainsHint()` rather than declaration.

Authored by flherne on Nov 29 2016, 12:52 AM.



Getting the docstring first allows us to skip searching for hints when there isn't one.

(It's also more obvious what's happening).

Test Plan

No behaviour changes, no test regressions.

Diff Detail

R53 KDevelop: Python Support
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.
flherne updated this revision to Diff 8581.Nov 29 2016, 12:52 AM
flherne retitled this revision from to Pass docstring into `docstringContainsHint()` rather than declaration..
flherne updated this object.
flherne edited the test plan for this revision. (Show Details)
flherne added a reviewer: brauch.
flherne set the repository for this revision to R53 KDevelop: Python Support.
Restricted Application added a subscriber: kdevelop-devel. · View Herald TranscriptNov 29 2016, 12:52 AM
brauch accepted this revision.Nov 29 2016, 11:12 PM
brauch edited edge metadata.

Thanks, makes sense.

This revision is now accepted and ready to land.Nov 29 2016, 11:12 PM
This revision was automatically updated to reflect the committed changes.