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

emitError for undefined sequences

Browse Source
This commit is contained in:
Matthias Mailänder
2015-08-29 08:18:52 +02:00
parent 589cb4cd0a
commit 5fc8a413e6
1 changed files with 8 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
OpenRA.Mods.Common/Lint/CheckSequences.cs
View File

@@ -20,7 +20,7 @@ namespace OpenRA.Mods.Common.Lint
{ {
class CheckSequences : ILintPass class CheckSequences : ILintPass
{ {
Action<string> emitWarning; Action<string> emitError;
List<MiniYamlNode> sequenceDefinitions; List<MiniYamlNode> sequenceDefinitions;
@@ -29,7 +29,7 @@ namespace OpenRA.Mods.Common.Lint
if (map != null && !map.SequenceDefinitions.Any()) if (map != null && !map.SequenceDefinitions.Any())
return; return;
this.emitWarning = emitWarning; this.emitError = emitError;
var sequenceSource = map != null ? map.SequenceDefinitions : new List<MiniYamlNode>(); var sequenceSource = map != null ? map.SequenceDefinitions : new List<MiniYamlNode>();
sequenceDefinitions = MiniYaml.MergeLiberal(sequenceSource, sequenceDefinitions = MiniYaml.MergeLiberal(sequenceSource,
@@ -49,7 +49,7 @@ namespace OpenRA.Mods.Common.Lint
{ {
var image = renderInfo.GetImage(actorInfo.Value, sequenceProvider, faction); var image = renderInfo.GetImage(actorInfo.Value, sequenceProvider, faction);
if (sequenceDefinitions.All(s => s.Key != image.ToLowerInvariant()) && !actorInfo.Value.Name.Contains("^")) if (sequenceDefinitions.All(s => s.Key != image.ToLowerInvariant()) && !actorInfo.Value.Name.Contains("^"))
emitWarning("Sprite image {0} from actor {1} using faction {2} has no sequence definition." emitError("Sprite image {0} from actor {1} using faction {2} has no sequence definition."
.F(image, actorInfo.Value.Name, faction)); .F(image, actorInfo.Value.Name, faction));
} }
} }
@@ -83,7 +83,7 @@ namespace OpenRA.Mods.Common.Lint
foreach (var imageOverride in LintExts.GetFieldValues(traitInfo, imageField, emitError)) foreach (var imageOverride in LintExts.GetFieldValues(traitInfo, imageField, emitError))
{ {
if (!string.IsNullOrEmpty(imageOverride) && sequenceDefinitions.All(s => s.Key != imageOverride.ToLowerInvariant())) if (!string.IsNullOrEmpty(imageOverride) && sequenceDefinitions.All(s => s.Key != imageOverride.ToLowerInvariant()))
emitWarning("Custom sprite image {0} from actor {1} has no sequence definition.".F(imageOverride, actorInfo.Value.Name)); emitError("Custom sprite image {0} from actor {1} has no sequence definition.".F(imageOverride, actorInfo.Value.Name));
else else
CheckDefintions(imageOverride, sequenceReference, actorInfo, sequence, faction, field, traitInfo); CheckDefintions(imageOverride, sequenceReference, actorInfo, sequence, faction, field, traitInfo);
} }
@@ -132,9 +132,9 @@ namespace OpenRA.Mods.Common.Lint
{ {
var definitions = sequenceDefinitions.FirstOrDefault(n => n.Key == imageOverride.ToLowerInvariant()); var definitions = sequenceDefinitions.FirstOrDefault(n => n.Key == imageOverride.ToLowerInvariant());
if (definitions == null) if (definitions == null)
emitWarning("Can't find sequence definition for projectile image {0} at weapon {1}.".F(imageOverride, weaponInfo.Key)); emitError("Can't find sequence definition for projectile image {0} at weapon {1}.".F(imageOverride, weaponInfo.Key));
else if (!definitions.Value.Nodes.Any(n => n.Key == sequence)) else if (!definitions.Value.Nodes.Any(n => n.Key == sequence))
emitWarning("Projectile sprite image {0} from weapon {1} does not define sequence {2} from field {3} of {4}" emitError("Projectile sprite image {0} from weapon {1} does not define sequence {2} from field {3} of {4}"
.F(imageOverride, weaponInfo.Key, sequence, field.Name, projectileInfo)); .F(imageOverride, weaponInfo.Key, sequence, field.Name, projectileInfo));
} }
} }
@@ -156,12 +156,12 @@ namespace OpenRA.Mods.Common.Lint
if (sequenceReference != null && sequenceReference.Prefix) if (sequenceReference != null && sequenceReference.Prefix)
{ {
if (!definitions.Value.Nodes.Any(n => n.Key.StartsWith(sequence))) if (!definitions.Value.Nodes.Any(n => n.Key.StartsWith(sequence)))
emitWarning("Sprite image {0} from actor {1} of faction {2} does not define sequence prefix {3} from field {4} of {5}" emitError("Sprite image {0} from actor {1} of faction {2} does not define sequence prefix {3} from field {4} of {5}"
.F(image, actorInfo.Value.Name, faction, sequence, field.Name, traitInfo)); .F(image, actorInfo.Value.Name, faction, sequence, field.Name, traitInfo));
} }
else if (definitions.Value.Nodes.All(n => n.Key != sequence)) else if (definitions.Value.Nodes.All(n => n.Key != sequence))
{ {
emitWarning("Sprite image {0} from actor {1} of faction {2} does not define sequence {3} from field {4} of {5}" emitError("Sprite image {0} from actor {1} of faction {2} does not define sequence {3} from field {4} of {5}"
.F(image, actorInfo.Value.Name, faction, sequence, field.Name, traitInfo)); .F(image, actorInfo.Value.Name, faction, sequence, field.Name, traitInfo));
} }
} }