Add a lint pass to check chrome logic validity.
This commit is contained in:
53
OpenRA.Mods.Common/Lint/CheckChromeLogic.cs
Normal file
53
OpenRA.Mods.Common/Lint/CheckChromeLogic.cs
Normal file
@@ -0,0 +1,53 @@
|
|||||||
|
#region Copyright & License Information
|
||||||
|
/*
|
||||||
|
* Copyright 2007-2015 The OpenRA Developers (see AUTHORS)
|
||||||
|
* This file is part of OpenRA, which is free software. It is made
|
||||||
|
* available to you under the terms of the GNU General Public License
|
||||||
|
* as published by the Free Software Foundation. For more information,
|
||||||
|
* see COPYING.
|
||||||
|
*/
|
||||||
|
#endregion
|
||||||
|
|
||||||
|
using System;
|
||||||
|
using System.Collections.Generic;
|
||||||
|
using System.Linq;
|
||||||
|
using System.Reflection;
|
||||||
|
using OpenRA.Traits;
|
||||||
|
using OpenRA.Widgets;
|
||||||
|
|
||||||
|
namespace OpenRA.Mods.Common.Lint
|
||||||
|
{
|
||||||
|
class CheckChromeLogic : ILintPass
|
||||||
|
{
|
||||||
|
public void Run(Action<string> emitError, Action<string> emitWarning)
|
||||||
|
{
|
||||||
|
foreach (var filename in Game.ModData.Manifest.ChromeLayout)
|
||||||
|
CheckInner(MiniYaml.FromFile(filename), filename, emitError);
|
||||||
|
}
|
||||||
|
|
||||||
|
void CheckInner(List<MiniYamlNode> nodes, string filename, Action<string> emitError)
|
||||||
|
{
|
||||||
|
foreach (var node in nodes)
|
||||||
|
{
|
||||||
|
if (node.Value == null)
|
||||||
|
continue;
|
||||||
|
|
||||||
|
if (node.Key == "Logic")
|
||||||
|
{
|
||||||
|
var typeNames = FieldLoader.GetValue<string[]>(node.Key, node.Value.Value);
|
||||||
|
foreach (var typeName in typeNames)
|
||||||
|
{
|
||||||
|
var type = Game.ModData.ObjectCreator.FindType(typeName);
|
||||||
|
if (type == null)
|
||||||
|
emitError("{0} refers to a logic object `{1}` that does not exist".F(filename, typeName));
|
||||||
|
else if (!typeof(ChromeLogic).IsAssignableFrom(type))
|
||||||
|
emitError("{0} refers to a logic object `{1}` that does not inherit from ChromeLogic".F(filename, typeName));
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
if (node.Value.Nodes != null)
|
||||||
|
CheckInner(node.Value.Nodes, filename, emitError);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
@@ -704,6 +704,7 @@
|
|||||||
<Compile Include="Scripting\Global\LightingGlobal.cs" />
|
<Compile Include="Scripting\Global\LightingGlobal.cs" />
|
||||||
<Compile Include="Traits\SupportPowers\ProduceActorPower.cs" />
|
<Compile Include="Traits\SupportPowers\ProduceActorPower.cs" />
|
||||||
<Compile Include="Widgets\Logic\GlobalChatLogic.cs" />
|
<Compile Include="Widgets\Logic\GlobalChatLogic.cs" />
|
||||||
|
<Compile Include="Lint\CheckChromeLogic.cs" />
|
||||||
</ItemGroup>
|
</ItemGroup>
|
||||||
<Import Project="$(MSBuildToolsPath)\Microsoft.CSharp.targets" />
|
<Import Project="$(MSBuildToolsPath)\Microsoft.CSharp.targets" />
|
||||||
<PropertyGroup>
|
<PropertyGroup>
|
||||||
|
|||||||
Reference in New Issue
Block a user