search

ideaconsult / jna-inchi

Wrapper to access InChI from Java
GNU Lesser General Public License v2.1
0 stars 0 forks source link

change switch statements to include default case if applicable #39

Closed uli-f closed 1 year ago

uli-f commented 1 year ago 
switch (parity) {
    case ODD:
        return InchiStereoParity.EVEN;
    case EVEN:
       return InchiStereoParity.ODD;
  }
return parity;

Can be written equivalently, but more explicitly as:

switch (parity) {
    case ODD:
        return InchiStereoParity.EVEN;
    case EVEN:
        return InchiStereoParity.ODD;
    default:
       return parity;
}
uli-f commented 1 year ago

Opened PR #44

ntk73 commented 1 year ago

PR #44 merged and closed