make mstoolkit (more) thread safe

[rickhelmus]

Hello,

I was hoping to use mstoolkit in a multi-threaded (with a private MSReader object per thread), but ran into issues. So far, I noticed the following needs to be changed:

• Use strtok_s() (Windows) or strtok_r() instead of strtok(), as the latter uses a static buffer and is therefore not thread safe. See e.g. https://stackoverflow.com/a/54702249
• The lLastScanIndex variable defined in mzParser.h is static. Moving this to a member variable of RAMPFILE seems like a proper solution.

So far, I quickly patched the above changes, and things seem OK. But at the moment I am also only using just the mzML part of the library. If more issues will arise, I will update this issue again.

Thanks,
Rick

edit1: I made the issue more generic for threading after noticing more issues.

[mhoopmann]

Hi Rick,
Yes, I should make MSToolkit more thread safe. I can implement changes where you suggest and look for others as well.
Cheers,
Mike