[SNMP4J] UsmUserEntry.java question

[kenping]

Hi,
    Has there a bug in UsmUserEntry.java at line 135 ?:

    if ((engineID != null) && (other.engineID != null)) {
      engineID.compareTo(other.engineID);   <----- ???
    }
    else if ((engineID != null) && (other.engineID != null)) {
      result = 1;
    }

    I changed it to follows:

    if ((engineID != null) && (other.engineID != null)) {
      result = engineID.compareTo(other.engineID);
    }
    else if ((engineID != null) && (other.engineID == null)) {
      result = 1;
    }
    else if ((engineID == null) && (other.engineID != null)) {
      result = -1;
    }

    Is it correct?

thanks for all.
yours kenping