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

Fix CA1012

Browse Source
This commit is contained in:
RoosterDragon
2023-03-12 11:16:15 +00:00
committed by abcdefg30
parent 19fa03435b
commit f336a956cf
19 changed files with 29 additions and 22 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
.editorconfig
View File

@@ -616,6 +616,9 @@ dotnet_code_quality.api_surface = all
#dotnet_code_quality.CA1010.additional_required_generic_interfaces =
dotnet_diagnostic.CA1010.severity = warning
# Abstract types should not have public constructors.
dotnet_diagnostic.CA1012.severity = warning
# Mark attributes with 'AttributeUsageAttribute'.
dotnet_diagnostic.CA1018.severity = warning

2
OpenRA.Game/Activities/Activity.cs
View File

@@ -86,7 +86,7 @@ namespace OpenRA.Activities
bool firstRunCompleted;
bool lastRun;
public Activity()
protected Activity()
{
IsInterruptible = true;
ChildHasPriority = true;

4
OpenRA.Game/Map/CellLayerBase.cs
View File

@@ -24,10 +24,10 @@ namespace OpenRA
protected readonly T[] Entries;
protected readonly Rectangle Bounds;
public CellLayerBase(Map map)
protected CellLayerBase(Map map)
: this(map.Grid.Type, new Size(map.MapSize.X, map.MapSize.Y)) { }
public CellLayerBase(MapGridType gridType, Size size)
protected CellLayerBase(MapGridType gridType, Size size)
{
Size = size;
Bounds = new Rectangle(0, 0, Size.Width, Size.Height);

9
OpenRA.Game/Scripting/ScriptContext.cs
View File

@@ -50,7 +50,8 @@ namespace OpenRA.Scripting
{
protected readonly Actor Self;
protected readonly ScriptContext Context;
public ScriptActorProperties(ScriptContext context, Actor self)
protected ScriptActorProperties(ScriptContext context, Actor self)
{
Self = self;
Context = context;
@@ -61,7 +62,8 @@ namespace OpenRA.Scripting
{
protected readonly Player Player;
protected readonly ScriptContext Context;
public ScriptPlayerProperties(ScriptContext context, Player player)
protected ScriptPlayerProperties(ScriptContext context, Player player)
{
Player = player;
Context = context;
@@ -87,7 +89,8 @@ namespace OpenRA.Scripting
protected override string MemberNotFoundError(string memberName) { return $"Table '{Name}' does not define a property '{memberName}'"; }
public readonly string Name;
public ScriptGlobal(ScriptContext context)
protected ScriptGlobal(ScriptContext context)
: base(context)
{
// GetType resolves the actual (subclass) type

2
OpenRA.Game/Scripting/ScriptObjectWrapper.cs
View File

@@ -28,7 +28,7 @@ namespace OpenRA.Scripting
readonly List<string> membersToRemove = new List<string>();
#endif
public ScriptObjectWrapper(ScriptContext context)
protected ScriptObjectWrapper(ScriptContext context)
{
Context = context;
}

2
OpenRA.Game/Support/VariableExpression.cs
View File

@@ -576,7 +576,7 @@ namespace OpenRA.Support
}
}
public VariableExpression(string expression)
protected VariableExpression(string expression)
{
Expression = expression;
}

5
OpenRA.Game/Widgets/Widget.cs
View File

@@ -204,9 +204,10 @@ namespace OpenRA.Widgets
public Rectangle Bounds;
public Widget Parent = null;
public Func<bool> IsVisible;
public Widget() { IsVisible = () => Visible; }
public Widget(Widget widget)
protected Widget() { IsVisible = () => Visible; }
protected Widget(Widget widget)
{
Id = widget.Id;
X = widget.X;

2
OpenRA.Mods.Common/Activities/Move/Move.cs
View File

@@ -372,7 +372,7 @@ namespace OpenRA.Mods.Common.Activities
readonly int terrainOrientationMargin;
protected int progress;
public MovePart(Move move, WPos from, WPos to, WAngle fromFacing, WAngle toFacing,
protected MovePart(Move move, WPos from, WPos to, WAngle fromFacing, WAngle toFacing,
WRot? fromTerrainOrientation, WRot? toTerrainOrientation, int terrainOrientationMargin,
int carryoverProgress, bool shouldArc, bool movingOnGroundLayer)
{

2
OpenRA.Mods.Common/Orders/GlobalButtonOrderGenerator.cs
View File

@@ -20,7 +20,7 @@ namespace OpenRA.Mods.Common.Orders
{
readonly string order;
public GlobalButtonOrderGenerator(string order)
protected GlobalButtonOrderGenerator(string order)
{
this.order = order;
}

2
OpenRA.Mods.Common/Orders/UnitOrderTargeter.cs
View File

@@ -20,7 +20,7 @@ namespace OpenRA.Mods.Common.Orders
readonly string cursor;
readonly bool targetEnemyUnits, targetAllyUnits;
public UnitOrderTargeter(string order, int priority, string cursor, bool targetEnemyUnits, bool targetAllyUnits)
protected UnitOrderTargeter(string order, int priority, string cursor, bool targetEnemyUnits, bool targetAllyUnits)
{
OrderID = order;
OrderPriority = priority;

2
OpenRA.Mods.Common/Traits/AffectsShroud.cs
View File

@@ -54,7 +54,7 @@ namespace OpenRA.Mods.Common.Traits
protected abstract void AddCellsToPlayerShroud(Actor self, Player player, PPos[] uv);
protected abstract void RemoveCellsFromPlayerShroud(Actor self, Player player);
public AffectsShroud(AffectsShroudInfo info)
protected AffectsShroud(AffectsShroudInfo info)
: base(info)
{
if (Info.Type == VisibilityType.Footprint)

2
OpenRA.Mods.Common/Traits/Attack/AttackBase.cs
View File

@@ -85,7 +85,7 @@ namespace OpenRA.Mods.Common.Traits
bool wasAiming;
public AttackBase(Actor self, AttackBaseInfo info)
protected AttackBase(Actor self, AttackBaseInfo info)
: base(info)
{
this.self = self;

2
OpenRA.Mods.Common/Traits/Conditions/ConditionalTrait.cs
View File

@@ -52,7 +52,7 @@ namespace OpenRA.Mods.Common.Traits
[Sync]
public bool IsTraitDisabled { get; private set; }
public ConditionalTrait(InfoType info)
protected ConditionalTrait(InfoType info)
{
Info = info;

2
OpenRA.Mods.Common/Traits/Conditions/GrantConditionOnLayer.cs
View File

@@ -26,7 +26,7 @@ namespace OpenRA.Mods.Common.Traits
protected readonly byte ValidLayerType;
protected int conditionToken = Actor.InvalidConditionToken;
public GrantConditionOnLayer(InfoType info, byte validLayer)
protected GrantConditionOnLayer(InfoType info, byte validLayer)
: base(info)
{
ValidLayerType = validLayer;

2
OpenRA.Mods.Common/Traits/Render/SelectionDecorationsBase.cs
View File

@@ -31,7 +31,7 @@ namespace OpenRA.Mods.Common.Traits.Render
DeveloperMode developerMode;
public SelectionDecorationsBase(SelectionDecorationsBaseInfo info)
protected SelectionDecorationsBase(SelectionDecorationsBaseInfo info)
{
Info = info;
}

2
OpenRA.Mods.Common/Traits/Render/WithDecorationBase.cs
View File

@@ -61,7 +61,7 @@ namespace OpenRA.Mods.Common.Traits.Render
int2 conditionalOffset;
BlinkState[] blinkPattern;
public WithDecorationBase(Actor self, InfoType info)
protected WithDecorationBase(Actor self, InfoType info)
: base(info)
{
Self = self;

2
OpenRA.Mods.Common/Traits/SupportPowers/SupportPower.cs
View File

@@ -141,7 +141,7 @@ namespace OpenRA.Mods.Common.Traits
[Desc("Sort order for the support power palette. Smaller numbers are presented earlier.")]
public readonly int SupportPowerPaletteOrder = 9999;
public SupportPowerInfo() { OrderName = GetType().Name + "Order"; }
protected SupportPowerInfo() { OrderName = GetType().Name + "Order"; }
}
public class SupportPower : PausableConditionalTrait<SupportPowerInfo>

2
OpenRA.Mods.Common/TraitsInterfaces.cs
View File

@@ -569,7 +569,7 @@ namespace OpenRA.Mods.Common.Traits
public readonly string Name;
public readonly int DisplayOrder;
public EditorActorOption(string name, int displayOrder)
protected EditorActorOption(string name, int displayOrder)
{
Name = name;
DisplayOrder = displayOrder;

2
OpenRA.Mods.Common/Widgets/Logic/Editor/CommonSelectorLogic.cs
View File

@@ -46,7 +46,7 @@ namespace OpenRA.Mods.Common.Widgets.Logic
protected string[] allCategories;
protected string searchFilter;
public CommonSelectorLogic(Widget widget, ModData modData, World world, WorldRenderer worldRenderer, string templateListId, string previewTemplateId)
protected CommonSelectorLogic(Widget widget, ModData modData, World world, WorldRenderer worldRenderer, string templateListId, string previewTemplateId)
{
Widget = widget;
ModData = modData;