FreeMind / Patches / #145 Fixed unnecessary check

#145 Fixed unnecessary check

Status: open-fixed

Owner: Christian Foltin

Labels: None

Priority: 5

Updated: 2014-08-25

Created: 2011-12-11

Creator: Anonymous

Private: No

noteViewerComponent is checked here to see whether it is null, but this value can't be null because it was previously dereferenced and if it were null a null pointer exception would have occurred at the earlier dereference. Essentially, this code and the previous dereference disagree as to whether this value is allowed to be null.

Discussion

Comment has been marked as spam.
Undo

View and moderate all "patches Discussion" comments posted by this user

Mark all as spam, and block user from posting to "Patches"

Anonymous - 2011-12-11

Last edit: Anonymous 2014-07-20

Add attachments
Cancel
You seem to have CSS turned off. Please don't fill out this field.

You seem to have CSS turned off. Please don't fill out this field.

New Attachment:

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Christian Foltin - 2012-02-05

commit 90450053c0372e0071c16b090e2e84aa8d51f83a

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Christian Foltin - 2012-02-05

assigned_to: nobody --> christianfoltin

status: open --> open-fixed
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Christian Foltin - 2012-02-05

Dear contributor,

thanks for your patch that has been integrated into the version control. You can verify that this is correct by considering
http://freemind.git.sourceforge.net/git/gitweb.cgi?p=freemind/freemind;a=summary

Best regards,

Chris from the FreeMind team

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Fixed unnecessary check

A premier mind-mapping software written in Java

Group

Searches

Help

#145 Fixed unnecessary check

Discussion