I'm still just learning vala so I could be wrong, but I think Coverity is correct.
The vala code that generated the C that Coverity is warning is:
public void alter_label (string new_title) { if (new_title == null) return; this.property_set (MENUITEM_PLAYER_NAME, new_title); }
I think "new_title" isn't allowed to be null because it's not prefixed with '?' in the argument list.
Removing the "if (new_title == null) return;" line removes the C code that generated the Coverity warning.
I'm still just learning vala so I could be wrong, but I think Coverity is correct.
The vala code that generated the C that Coverity is warning is:
public void alter_label (string new_title) property_ set (MENUITEM_ PLAYER_ NAME, new_title);
{
if (new_title == null) return;
this.
}
I think "new_title" isn't allowed to be null because it's not prefixed with '?' in the argument list.
Removing the "if (new_title == null) return;" line removes the C code that generated the Coverity warning.