epic/OpenRA
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #11153 from RoosterDragon/selection-fast-checks

Browse Source 
Use Contains method on Selection when possible.
This commit is contained in:
abcdefg30
2016-04-20 17:50:45 +02:00
parent d7f140d5a3 178fcfaf8c
commit 7a13fb189c
3 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
OpenRA.Game/Graphics/WorldRenderer.cs
View File

@@ -192,7 +192,7 @@ namespace OpenRA.Graphics
{
foreach (var g in World.ActorsHavingTrait<Selectable>().Where(a => !a.Disposed
&& !World.FogObscures(a)
&& !World.Selection.Actors.Contains(a)))
&& !World.Selection.Contains(a)))
{
if (Game.Settings.Game.StatusBars == StatusBarsType.Standard)
new SelectionBarsRenderable(g, false, false).Render(this);

2
OpenRA.Mods.Common/Effects/RallyPointIndicator.cs
View File

@@ -81,7 +81,7 @@ namespace OpenRA.Mods.Common.Effects
if (!building.IsInWorld || !building.Owner.IsAlliedWith(building.World.LocalPlayer))
return SpriteRenderable.None;
if (!building.World.Selection.Actors.Contains(building))
if (!building.World.Selection.Contains(building))
return SpriteRenderable.None;
return RenderInner(wr);

4
OpenRA.Mods.Common/Widgets/WorldCommandWidget.cs
View File

@@ -215,7 +215,7 @@ namespace OpenRA.Mods.Common.Widgets
}
var next = bases
.SkipWhile(b => !world.Selection.Actors.Contains(b))
.SkipWhile(b => !world.Selection.Contains(b))
.Skip(1)
.FirstOrDefault();
@@ -240,7 +240,7 @@ namespace OpenRA.Mods.Common.Widgets
return true;
var next = facilities
.SkipWhile(b => !world.Selection.Actors.Contains(b))
.SkipWhile(b => !world.Selection.Contains(b))
.Skip(1)
.FirstOrDefault();