Review Request 111135: Handle return cmake statement

Commit Hook null at kde.org
Wed Jun 19 23:55:22 UTC 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111135/
-----------------------------------------------------------

(Updated June 19, 2013, 11:55 p.m.)


Status
------

This change has been marked as submitted.


Review request for KDevelop.


Description
-------

Handle return statement when walking throuth a function or macro. Recursive cmake macro using return to stop recursion leads to segfault due to stackoverflow in Kdevelop without proper support of this keyword.

Fixes crash from the bug https://bugs.kde.org/show_bug.cgi?id=320875


Diffs
-----

  projectmanagers/cmake/parser/cmakeprojectvisitor.h cc314fd 
  projectmanagers/cmake/parser/cmakeprojectvisitor.cpp 5299b7c 

Diff: http://git.reviewboard.kde.org/r/111135/diff/


Testing
-------

Manually tested


File Attachments
----------------

Formatted patch
  http://git.reviewboard.kde.org/media/uploaded/files/2013/06/19/0002-Support-return-statement.patch


Thanks,

Sergey Vidyuk

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20130619/3a49c68a/attachment.html>


More information about the KDevelop-devel mailing list