6.0 Condition check issue

The following code, after upgrading from 5 to 6.0 and then 6.01 is throwing a warning on the if (changed) line towards the end. It says that changed is always false but that is not true. The various checks along the way can set changed to true.

 curgroups = gwitem.getGroups();
                if (curgroups != groups) {
                    try {
                        gwitem.setGroups(groups);
                        changed = true;
                    }
                    catch (Exception ee) {
                        // Oooookay, ignore then.
                    }
                }
                if (changed) {
                    roster.updateRosterItem(gwitem);
                }
]]>

2 comments

If you change the type of changed to boolean, instead of Boolean, the warning is removed. However, this is still a bug and you should submit a JIRA issue about this if it is important to you.

Bas

0

Oh sure, I can do that. =) I'm used to not being able to submit JIRA issues myself. ;) (typically I see 'you must post in the forum and we'll create JIRA issues) Thanks! Will do. IDEA-9996

0

Please sign in to leave a comment.