D5915: Infer correct type from 'and' / 'or' expressions

Nicolás Alvarez noreply at phabricator.kde.org
Sat May 20 00:54:21 UTC 2017


nalvarez marked 2 inline comments as done.
nalvarez added inline comments.

INLINE COMMENTS

> flherne wrote in expressionvisitor.cpp:736
> Nitpick - convention for these is either just `v` or `operandVisitor` (the latter usually when then there's more than one). I don't have a strong opinion.

Renamed, mainly for consistency with the other methods which use `v`.

> flherne wrote in pyduchaintest.cpp:510
> Nitpick - you can use single quotes for the Python strings, instead of backslashed double-quotes. They're semantically identical.

Good point! Changed.

REPOSITORY
  R53 KDevelop: Python Support

BRANCH
  master

REVISION DETAIL
  https://phabricator.kde.org/D5915

To: nalvarez, brauch, flherne
Cc: flherne, kdevelop-devel, geetamc, Pilzschaf, akshaydeo, surgenight, arrowdodger
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20170520/02509e0c/attachment.html>


More information about the KDevelop-devel mailing list