Addition of checks and use of stdlib_experimental_ascii

[jvdp1]

Additions/changes:

• some checks in parse_mode to return errors for modes such as rr, rw, r++t, rbt. (Note: I couldn't add these tests in CMake, or it should be one per file)
• the function open can return the iostat if requested
• (@certik) since Place ASCII control characters in derived type  #49 has been closed, whitechar is now replaced by is_blank

@scivision : Because I added checks in the function parse_mode, it uses again if statements instead of select case. Good for you?

[scivision]

yes I think it's fine/necessary to use if here with the more complex conditions now present

[certik]

Thank you @jvdp1 for doing this, it looks really good. I left some comments regarding the API and testing.

[jvdp1]

Thank you @jvdp1 for doing this, it looks really good. I left some comments regarding the API and testing.

Thanks for the review.

[certik]

In your last commit, in the tests I think you forgot to add io in the 3rd test from last. (I am on my phone, so I can't comment there directly.)

…

On Sun, Jan 5, 2020, at 9:44 AM, Jeremie Vandenplas wrote: > > Thank you @jvdp1 <https://github.com/jvdp1> for doing this, it looks really good. I left some comments regarding the API and testing. Thanks for the review. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#82?email_source=notifications&email_token=AAAFAWAMZPEYHRA6CLBHG4TQ4IFABA5CNFSM4KC2LVWKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEID2WXQ#issuecomment-570927966>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAAFAWC5PDMQAP4A7WCAJSDQ4IFABANCNFSM4KC2LVWA>.

[jvdp1]

In your last commit, in the tests I think you forgot to add io in the 3rd test from last. (I am on my phone, so I can't comment there directly.)

Resolved!

[certik]

Perfect! I think it looks good now. +1 to merge. Go ahead and merge it if you want.

…

On Sun, Jan 5, 2020, at 10:35 AM, Jeremie Vandenplas wrote: > > In your last commit, in the tests I think you forgot to add io in the 3rd test from last. (I am on my phone, so I can't comment there directly.) Resolved! — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#82?email_source=notifications&email_token=AAAFAWBSWCWM3CRHCDSH4TDQ4IK6DA5CNFSM4KC2LVWKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEID3UBA#issuecomment-570931716>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAAFAWFV7PSVGON5DXLHNRDQ4IK6DANCNFSM4KC2LVWA>.