Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Handle new naming inspections on certain enums #28051

Merged
merged 8 commits into from
May 2, 2024
Merged

Handle new naming inspections on certain enums #28051

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
981a19f
Disable naming inspections in p/invoke code
frenzibyte May 1, 2024
02be275
Disable naming inspections in country/language enums
frenzibyte May 1, 2024
2fd8950
Disable inconsistent naming in some fields of `LegacyManiaSkinConfigu…
frenzibyte May 1, 2024
97fc2a5
Disable inconsistent naming in some fields of `ScoreRank`
frenzibyte May 1, 2024
16bae4f
Update naming of enum fields in `StartMode`
frenzibyte May 1, 2024
e8a6381
Update naming of enum fields in `ObjType`
frenzibyte May 1, 2024
9dc1a58
Add few abbreviations from existing enums to appease naming inspection
frenzibyte May 1, 2024
30fd40e
Avoid disable/restore pairs
frenzibyte May 1, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 6 additions & 0 deletions osu.Desktop/NVAPI.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -489,6 +489,7 @@ internal struct NvApplication
public static uint Stride => (uint)Marshal.SizeOf(typeof(NvApplication)) | (2 << 16);
}

[SuppressMessage("ReSharper", "InconsistentNaming")]
internal enum NvStatus
{
OK = 0, // Success. Request is completed.
Expand Down Expand Up @@ -611,20 +612,23 @@ internal enum NvStatus
FIRMWARE_REVISION_NOT_SUPPORTED = -200, // The device's firmware is not supported.
}

[SuppressMessage("ReSharper", "InconsistentNaming")]
internal enum NvSystemType
{
UNKNOWN = 0,
LAPTOP = 1,
DESKTOP = 2
}

[SuppressMessage("ReSharper", "InconsistentNaming")]
internal enum NvGpuType
{
UNKNOWN = 0,
IGPU = 1, // Integrated
DGPU = 2, // Discrete
}

[SuppressMessage("ReSharper", "InconsistentNaming")]
internal enum NvSettingID : uint
{
OGL_AA_LINE_GAMMA_ID = 0x2089BF6C,
Expand Down Expand Up @@ -717,6 +721,7 @@ internal enum NvSettingID : uint
INVALID_SETTING_ID = 0xFFFFFFFF
}

[SuppressMessage("ReSharper", "InconsistentNaming")]
internal enum NvShimSetting : uint
{
SHIM_RENDERING_MODE_INTEGRATED = 0x00000000,
Expand All @@ -731,6 +736,7 @@ internal enum NvShimSetting : uint
SHIM_RENDERING_MODE_DEFAULT = SHIM_RENDERING_MODE_AUTO_SELECT
}

[SuppressMessage("ReSharper", "InconsistentNaming")]
internal enum NvThreadControlSetting : uint
{
OGL_THREAD_CONTROL_ENABLE = 0x00000001,
Expand Down
3 changes: 3 additions & 0 deletions osu.Desktop/Windows/WindowsAssociationManager.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@
// See the LICENCE file in the repository root for full licence text.

using System;
using System.Diagnostics.CodeAnalysis;
using System.IO;
using System.Runtime.InteropServices;
using System.Runtime.Versioning;
Expand Down Expand Up @@ -163,6 +164,7 @@ string getLocalisedString(LocalisableString s)
[DllImport("Shell32.dll")]
private static extern void SHChangeNotify(EventId wEventId, Flags uFlags, IntPtr dwItem1, IntPtr dwItem2);

[SuppressMessage("ReSharper", "InconsistentNaming")]
private enum EventId
{
/// <summary>
Expand All @@ -172,6 +174,7 @@ private enum EventId
SHCNE_ASSOCCHANGED = 0x08000000
}

[SuppressMessage("ReSharper", "InconsistentNaming")]
private enum Flags : uint
{
SHCNF_IDLIST = 0x0000
Expand Down
76 changes: 38 additions & 38 deletions osu.Game/IO/Legacy/SerializationReader.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -123,58 +123,58 @@ public object ReadObject()

switch (t)
{
case ObjType.boolType:
case ObjType.BoolType:
return ReadBoolean();

case ObjType.byteType:
case ObjType.ByteType:
return ReadByte();

case ObjType.uint16Type:
case ObjType.UInt16Type:
return ReadUInt16();

case ObjType.uint32Type:
case ObjType.UInt32Type:
return ReadUInt32();

case ObjType.uint64Type:
case ObjType.UInt64Type:
return ReadUInt64();

case ObjType.sbyteType:
case ObjType.SByteType:
return ReadSByte();

case ObjType.int16Type:
case ObjType.Int16Type:
return ReadInt16();

case ObjType.int32Type:
case ObjType.Int32Type:
return ReadInt32();

case ObjType.int64Type:
case ObjType.Int64Type:
return ReadInt64();

case ObjType.charType:
case ObjType.CharType:
return ReadChar();

case ObjType.stringType:
case ObjType.StringType:
return base.ReadString();

case ObjType.singleType:
case ObjType.SingleType:
return ReadSingle();

case ObjType.doubleType:
case ObjType.DoubleType:
return ReadDouble();

case ObjType.decimalType:
case ObjType.DecimalType:
return ReadDecimal();

case ObjType.dateTimeType:
case ObjType.DateTimeType:
return ReadDateTime();

case ObjType.byteArrayType:
case ObjType.ByteArrayType:
return ReadByteArray();

case ObjType.charArrayType:
case ObjType.CharArrayType:
return ReadCharArray();

case ObjType.otherType:
case ObjType.OtherType:
throw new IOException("Deserialization of arbitrary type is not supported.");

default:
Expand All @@ -185,25 +185,25 @@ public object ReadObject()

public enum ObjType : byte
{
nullType,
boolType,
byteType,
uint16Type,
uint32Type,
uint64Type,
sbyteType,
int16Type,
int32Type,
int64Type,
charType,
stringType,
singleType,
doubleType,
decimalType,
dateTimeType,
byteArrayType,
charArrayType,
otherType,
ILegacySerializableType
NullType,
BoolType,
ByteType,
UInt16Type,
UInt32Type,
UInt64Type,
SByteType,
Int16Type,
Int32Type,
Int64Type,
CharType,
StringType,
SingleType,
DoubleType,
DecimalType,
DateTimeType,
ByteArrayType,
CharArrayType,
OtherType,
LegacySerializableType
}
}
40 changes: 20 additions & 20 deletions osu.Game/IO/Legacy/SerializationWriter.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -34,11 +34,11 @@ public override void Write(string str)
{
if (str == null)
{
Write((byte)ObjType.nullType);
Write((byte)ObjType.NullType);
}
else
{
Write((byte)ObjType.stringType);
Write((byte)ObjType.StringType);
base.Write(str);
}
}
Expand Down Expand Up @@ -125,94 +125,94 @@ public void WriteObject(object obj)
{
if (obj == null)
{
Write((byte)ObjType.nullType);
Write((byte)ObjType.NullType);
}
else
{
switch (obj)
{
case bool boolObj:
Write((byte)ObjType.boolType);
Write((byte)ObjType.BoolType);
Write(boolObj);
break;

case byte byteObj:
Write((byte)ObjType.byteType);
Write((byte)ObjType.ByteType);
Write(byteObj);
break;

case ushort ushortObj:
Write((byte)ObjType.uint16Type);
Write((byte)ObjType.UInt16Type);
Write(ushortObj);
break;

case uint uintObj:
Write((byte)ObjType.uint32Type);
Write((byte)ObjType.UInt32Type);
Write(uintObj);
break;

case ulong ulongObj:
Write((byte)ObjType.uint64Type);
Write((byte)ObjType.UInt64Type);
Write(ulongObj);
break;

case sbyte sbyteObj:
Write((byte)ObjType.sbyteType);
Write((byte)ObjType.SByteType);
Write(sbyteObj);
break;

case short shortObj:
Write((byte)ObjType.int16Type);
Write((byte)ObjType.Int16Type);
Write(shortObj);
break;

case int intObj:
Write((byte)ObjType.int32Type);
Write((byte)ObjType.Int32Type);
Write(intObj);
break;

case long longObj:
Write((byte)ObjType.int64Type);
Write((byte)ObjType.Int64Type);
Write(longObj);
break;

case char charObj:
Write((byte)ObjType.charType);
Write((byte)ObjType.CharType);
base.Write(charObj);
break;

case string stringObj:
Write((byte)ObjType.stringType);
Write((byte)ObjType.StringType);
base.Write(stringObj);
break;

case float floatObj:
Write((byte)ObjType.singleType);
Write((byte)ObjType.SingleType);
Write(floatObj);
break;

case double doubleObj:
Write((byte)ObjType.doubleType);
Write((byte)ObjType.DoubleType);
Write(doubleObj);
break;

case decimal decimalObj:
Write((byte)ObjType.decimalType);
Write((byte)ObjType.DecimalType);
Write(decimalObj);
break;

case DateTime dateTimeObj:
Write((byte)ObjType.dateTimeType);
Write((byte)ObjType.DateTimeType);
Write(dateTimeObj);
break;

case byte[] byteArray:
Write((byte)ObjType.byteArrayType);
Write((byte)ObjType.ByteArrayType);
base.Write(byteArray);
break;

case char[] charArray:
Write((byte)ObjType.charArrayType);
Write((byte)ObjType.CharArrayType);
base.Write(charArray);
break;

Expand Down
2 changes: 2 additions & 0 deletions osu.Game/Localisation/Language.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,10 +2,12 @@
// See the LICENCE file in the repository root for full licence text.

using System.ComponentModel;
using System.Diagnostics.CodeAnalysis;
using JetBrains.Annotations;

namespace osu.Game.Localisation
{
[SuppressMessage("ReSharper", "InconsistentNaming")]
[UsedImplicitly(ImplicitUseTargetFlags.WithMembers)]
public enum Language
{
Expand Down
2 changes: 2 additions & 0 deletions osu.Game/Scoring/ScoreRank.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,7 @@ public enum ScoreRank

[LocalisableDescription(typeof(BeatmapsStrings), nameof(BeatmapsStrings.RankSH))]
[Description(@"S+")]
// ReSharper disable once InconsistentNaming
SH,

[LocalisableDescription(typeof(BeatmapsStrings), nameof(BeatmapsStrings.RankX))]
Expand All @@ -43,6 +44,7 @@ public enum ScoreRank

[LocalisableDescription(typeof(BeatmapsStrings), nameof(BeatmapsStrings.RankXH))]
[Description(@"SS+")]
// ReSharper disable once InconsistentNaming
XH,
}
}
8 changes: 4 additions & 4 deletions osu.Game/Screens/OnlinePlay/Multiplayer/Match/MultiplayerMatchSettingsOverlay.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -486,16 +486,16 @@ private enum StartMode
Off = 0,

[Description("30 seconds")]
Seconds_30 = 30,
Seconds30 = 30,

[Description("1 minute")]
Seconds_60 = 60,
Seconds60 = 60,

[Description("3 minutes")]
Seconds_180 = 180,
Seconds180 = 180,

[Description("5 minutes")]
Seconds_300 = 300
Seconds300 = 300
}
}
}
3 changes: 3 additions & 0 deletions osu.Game/Skinning/LegacyManiaSkinConfigurationLookup.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,10 @@ public enum LegacyManiaSkinConfigurationLookups
LeftStageImage,
RightStageImage,
BottomStageImage,

// ReSharper disable once InconsistentNaming
Hit300g,

Hit300,
Hit200,
Hit100,
Expand Down
2 changes: 2 additions & 0 deletions osu.Game/Users/CountryCode.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@
// See the LICENCE file in the repository root for full licence text.

using System.ComponentModel;
using System.Diagnostics.CodeAnalysis;
using JetBrains.Annotations;
using Newtonsoft.Json;
using Newtonsoft.Json.Converters;
Expand All @@ -12,6 +13,7 @@ namespace osu.Game.Users
/// Matches `osu_countries` database table.
/// </summary>
[JsonConverter(typeof(StringEnumConverter))]
[SuppressMessage("ReSharper", "InconsistentNaming")]
[UsedImplicitly(ImplicitUseTargetFlags.WithMembers)]
public enum CountryCode
{
Expand Down
Loading
Loading