Review Request 129599: segfault when ruby was not installed
Leslie Zhai
xiangzhai83 at gmail.com
Tue Dec 6 08:40:59 UTC 2016
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129599/
-----------------------------------------------------------
(Updated Dec. 6, 2016, 8:40 a.m.)
Status
------
This change has been marked as submitted.
Review request for KDevelop and Kevin Funk.
Changes
-------
Submitted with commit 5c330b1180cecaaaace54f47124496a2804e1ed1 by Leslie Zhai to branch master.
Repository: kdev-ruby
Description
-------
if ruby was not installed, ruby.readAllStandardOutput().split(' ')'s size is ZERO, so I simply added the check:
```
QByteArray allOutput = ruby.readAllStandardOutput();
if (allOutput.split(' ').size() > 1)
```
Diffs
-----
languagesupport.cpp d618d7d
Diff: https://git.reviewboard.kde.org/r/129599/diff/
Testing
-------
1. Not intalled ruby, kdevelop import redmin (a bug tracker written in ruby) will NOT segfault
2. Installed ruby, the same story
Thanks,
Leslie Zhai
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20161206/ce6815ff/attachment-0001.html>
More information about the KDevelop-devel
mailing list