search

ideaconsult / jna-inchi

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

removed timeout option as it is not picked up by the native RInChI li… #41

Closed uli-f closed 1 year ago

uli-f commented 1 year ago

…brary

ntk73 commented 1 year ago

@uli-f , The removal of timeout option broke the code in RInChIOptionParser in cdk-rinchi. I fixed this by temporary commenting out parsing case "W". We could delete it completely.

uli-f commented 1 year ago

The removal of timeout option broke the code in RInChIOptionParser in cdk-rinchi. I fixed this by temporary commenting out parsing case "W". We could delete it completely.

Thank you for carrying the change over to RInChIOptionParser in cdk-rinchi. Would you mind removing the commented out code in cdk-rinchi?

ntk73 commented 1 year ago

Thank you for carrying the change over to RInChIOptionParser in cdk-rinchi. Would you mind removing the commented out code in cdk-rinchi?

Done (see https://github.com/ideaconsult/cdk/commit/265e54c5075e6daaba27a63fe3034d7db658bbd3)