Fix a Wself-assign-field clang warning
The arg that should be on the rhs seems to be commented out for some reason, and assigning a variable to itself is pointless, so just comment it out with a comment explaining why.
This commit is contained in:
@@ -338,7 +338,11 @@ Dialog::Dialog(MotifUI *parent, char *title, char *base_directory,
|
|||||||
StringFree(pattern_string);
|
StringFree(pattern_string);
|
||||||
if (help)
|
if (help)
|
||||||
{
|
{
|
||||||
_help_callback_data = _help_callback_data;
|
// JET - this does nothing - should probably be assigned from
|
||||||
|
// help_callback_data, but that argument seems to have been
|
||||||
|
// removed from the arg list (commented out for some reason).
|
||||||
|
// So, I'll just comment this out for now:
|
||||||
|
// _help_callback_data = _help_callback_data;
|
||||||
XtAddCallback(_w, XmNhelpCallback, &Dialog::DialogCB, (XtPointer) this);
|
XtAddCallback(_w, XmNhelpCallback, &Dialog::DialogCB, (XtPointer) this);
|
||||||
}
|
}
|
||||||
else
|
else
|
||||||
|
|||||||
Reference in New Issue
Block a user