Remove use of strtok in CMSSW

[Dr15Jones]

The C function strtok is thread hostile as it internally holds static memory. It can relatively easily be replaced by calling strtok_r instead.

[Dr15Jones]

assign core

[cmsbuild]

New categories assigned: core

@Dr15Jones,@smuzaffar you have been requested to review this Pull request/Issue and eventually sign? Thanks

[cmsbuild]

A new Issue was created by @Dr15Jones Chris Jones.

@davidlange6, @Dr15Jones, @smuzaffar, @fabiocos, @kpedro88 can you please review it and eventually sign/assign? Thanks.

cms-bot commands are listed here

[Dr15Jones]

The following files make use of strtok:
Alignment/Geners/src/BinaryArchiveBase.cc
CalibTracker/SiStripChannelGain/plugins/SiStripGainFromAsciiFile.cc
DQMOffline/CalibTracker/plugins/SiStripQualityHotStripIdentifierRoot.cc
EventFilter/HcalRawToDigi/plugins/HcalLaserEventFilter2012.cc
EventFilter/L1TXRawToDigi/test/testUCTUnpacker.cpp
FWCore/Services/plugins/SimpleMemoryCheck.cc
IORawData/DTCommissioning/plugins/RawFile.cc
PhysicsTools/MVATrainer/plugins/mlp_gen.cc

[fabiocos]

#25642 implementing the replacement for SimpleMemoryChecker has been merged

[Dr15Jones]

The following pull requests remove all uses of strtok: #25654 #25655 #25656 #25657 #25658 #25660

[Dr15Jones]

+1

[cmsbuild]

This issue is fully signed and ready to be closed.