Does not parsing all players properly

[gdhsnlvr]

Lib does not parsing some players and event related to this players have null in player field. examples: https://www.hltv.org/matches/2320066/sk-vs-astralis-iem-katowice-2018 map 2 (de_mirage). https://www.hltv.org/matches/2320074/renegades-vs-cloud9-iem-katowice-2018 map 3 (de_train). parser.PlayingParticipants have 9 players, PlayerKilledEventArgs have null in fields Killer/Victim/Assister when event related to missing player. (version is last stable realise 1.2.0)

[InspektorLulz]

Hi I found that the library doesnt parse all entries in a Stringtable Packet. Just delete line 83 and 84 in Namespace: DemoInfo.DP.Handler Class: CreateStringTableUserInfoHandler .

if (userdata.Length == 0) break;

That at least solved my issue with players not beeing parsed.

[master117]

Deleting that will lead to unwated behavior, change it to:

if (userdata.Length == 0)
  continue;

instead.

Edit:

I just checked, it is already

if (userdata.Length == 0)
  continue;

where did you see:

if (userdata.Length == 0)
  break;

?

[master117]

I just removed

if (userdata.Length == 0)
  continue;

as you suggested. This leads to exceptions in Helper.cs:

		public static long ReadInt64SwapEndian(this BinaryReader reader)
		{
			return BitConverter.ToInt64(reader.ReadBytes(8).Reverse().ToArray(), 0);
		}

steaming from:

internal PlayerInfo(BinaryReader reader)

[InspektorLulz]

In the stable release 1.2.0 from November 2015 (you can download the zip) there is a break instead of continue. You are right, it should be continue. I didn't get the exception though.