fix for icc: change unsigned int to size_t when looping over vector elements

[smuzaffar]

change unsigned int to size_t to avoid strange icc optimization which causes segfault. This is the right thing to do any way

[cmsbuild]

A new Pull Request was created by @smuzaffar (Malik Shahzad Muzaffar) for CMSSW_5_3_X.

fix fro icc: change unsigned int to size_t when looping over vector elements

It involves the following packages:

PhysicsTools/MVAComputer

@nclopezo, @vadler, @cmsbuild, @Degano, @monttj can you please review it and eventually sign? Thanks.
You can sign-off by replying to this message having '+1' in the first line of your reply.
You can reject by replying to this message having '-1' in the first line of your reply.
@smuzaffar you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

[smuzaffar]

+tested

[davidlt]

Could you also attempt to generate a small testcase, pre-process it and send to Intel people?

[ktf]

Already done.

[smuzaffar]

icc has mange to reproduce it and problem is with optimization by inlining.
This is anyway the right fix, so bypassing it