Handle blank lines at the end of Karplus data file for jcoupling action

[drroe]

6.27.1.

Fixes behavior where CPPTRAJ would segfault if there were blank lines at the end of the Karplus data file in the jcoupling action.

[drroe]

@hainm any insight into the pytraj error here: https://github.com/Amber-MD/cpptraj/actions/runs/9685730413/job/26726487745?pr=1088

Could it be related to the recent changes? As far as I can tell this change shouldnt affect pytraj at all.

[hainm]

I think it does. (Guess that from reading the title). I will confirm this later (on the way home)). Hai

…

On Wed, Jun 26, 2024 at 4:17 PM Daniel R. Roe ***@***.***> wrote: @hainm <https://github.com/hainm> any insight into the pytraj error here: https://github.com/Amber-MD/cpptraj/actions/runs/9685730413/job/26726487745?pr=1088 Could it be related to the recent changes? As far as I can tell this change shouldnt affect pytraj at all. — Reply to this email directly, view it on GitHub <#1088 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABB645P57KPKC3I3JYPVEHTZJMOXJAVCNFSM6AAAAABJ6Q55BOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOJSGU2TENBXGM> . You are receiving this because you were mentioned.Message ID: ***@***.***>

[hainm]

I think it does. (Guess that from reading the title). I will confirm this later (on the way home)).

Oh, my mistake. It's because I did code refactoring and the code uses strenum

I will fix this.

In this pytraj's PR, the tests were run with latest python version. That's why tests didn't fail. Amber-MD/pytraj#1657

[hainm]

green now

[drroe]

Thanks Hai!