Remove redundant copy of BuildRequestData.TargetNames

[dfederm]

Minor change to remove an extra defensive copy of an array. A BuildRequestData ctor overload copies it despite the this ctor it calls also copying it.

Also a very minor perf change to use an array and copy via CopyTo instead of cloning with a List<>.

[Forgind]

Thanks!

Nit: BuildManager_Tests, line 1409 has data.TargetNames.ToArray(); with this change, you should be able to simplify that too.

[dfederm]

BuildManager_Tests, line 1409 has data.TargetNames.ToArray(); with this change, you should be able to simplify that too.

@Forgind BuildRequestData still exposes TargetNames as an ICollection<string>, so the alternative would be a cast, and I think it'd be better not to assume. You did make me realize something important though; ICollection is technically mutable, so changing to an array for the underlying type breaks that. I'll put it back at a List<>