D16143: Change null pointer reference from zero to nullptr

Chris Rizzitello noreply at phabricator.kde.org
Fri Oct 12 19:52:15 BST 2018


rizzitello requested changes to this revision.
rizzitello added inline comments.
This revision now requires changes to proceed.

INLINE COMMENTS

> mainwindow.cpp:139
>  {
> -	if (colorPicker == 0) {
> +	if (colorPicker) {
>  		colorPicker = new ColorPicker(this);

check your logic here. 
Pretty sure we want

  if(!colorPicker)

here

REPOSITORY
  R337 KTurtle

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

To: Shroudmaster, tcanabrava, rizzitello
Cc: rizzitello, kde-edu, narvaez, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20181012/54caa0f9/attachment-0001.html>


More information about the kde-edu mailing list