Skip to content

Commit

Permalink
Revert "Automated JetBrains cleanup"
Browse files Browse the repository at this point in the history
This reverts commit e9c3497.
  • Loading branch information
jinek committed Oct 22, 2024
1 parent 2dcb1c8 commit 34c9336
Show file tree
Hide file tree
Showing 64 changed files with 199 additions and 195 deletions.
2 changes: 1 addition & 1 deletion Directory.Core.Build.props
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@
<TreatWarningsAsErrors Condition=" '$(Configuration)' == 'RELEASE' ">true</TreatWarningsAsErrors>
</PropertyGroup>
<PropertyGroup>
<AvaloniaAccessUnstablePrivateApis>true</AvaloniaAccessUnstablePrivateApis>
<AvaloniaAccessUnstablePrivateApis>true</AvaloniaAccessUnstablePrivateApis>
<Avalonia_I_Want_To_Use_Private_Apis_In_Nuget_Package_And_Promise_To_Pin_The_Exact_Avalonia_Version_In_Package_Dependency>true</Avalonia_I_Want_To_Use_Private_Apis_In_Nuget_Package_And_Promise_To_Pin_The_Exact_Avalonia_Version_In_Package_Dependency>
</PropertyGroup>
</Project>
13 changes: 5 additions & 8 deletions src/Consolonia.Core/Drawing/ConsoloniaRenderInterface.cs
Original file line number Diff line number Diff line change
Expand Up @@ -90,14 +90,12 @@ public IWriteableBitmapImpl LoadWriteableBitmap(Stream stream)
throw new NotImplementedException();
}

IBitmapImpl IPlatformRenderInterface.LoadBitmapToWidth(Stream stream, int width,
BitmapInterpolationMode interpolationMode)
IBitmapImpl IPlatformRenderInterface.LoadBitmapToWidth(Stream stream, int width, BitmapInterpolationMode interpolationMode)
{
throw new NotImplementedException();
}

IBitmapImpl IPlatformRenderInterface.LoadBitmapToHeight(Stream stream, int height,
BitmapInterpolationMode interpolationMode)
IBitmapImpl IPlatformRenderInterface.LoadBitmapToHeight(Stream stream, int height, BitmapInterpolationMode interpolationMode)
{
throw new NotImplementedException();
}
Expand All @@ -107,7 +105,7 @@ IBitmapImpl IPlatformRenderInterface.ResizeBitmap(IBitmapImpl bitmapImpl, PixelS
{
throw new NotImplementedException();
}

public IBitmapImpl LoadBitmap(
PixelFormat format,
AlphaFormat alphaFormat,
Expand All @@ -119,11 +117,10 @@ public IBitmapImpl LoadBitmap(
throw new NotImplementedException();
}

public IGlyphRunImpl CreateGlyphRun(IGlyphTypeface glyphTypeface, double fontRenderingEmSize,
IReadOnlyList<GlyphInfo> glyphInfos,
public IGlyphRunImpl CreateGlyphRun(IGlyphTypeface glyphTypeface, double fontRenderingEmSize, IReadOnlyList<GlyphInfo> glyphInfos,
Point baselineOrigin)
{
return new GlyphRunImpl(glyphTypeface, glyphInfos, baselineOrigin);
return new GlyphRunImpl(glyphTypeface,glyphInfos,baselineOrigin);
}

public IPlatformRenderInterfaceContext CreateBackendContext(IPlatformGraphicsContext graphicsApiContext)
Expand Down
7 changes: 3 additions & 4 deletions src/Consolonia.Core/Drawing/DrawingContextImpl.cs
Original file line number Diff line number Diff line change
Expand Up @@ -170,7 +170,6 @@ public void DrawGlyphRun(IBrush foreground, IGlyphRunImpl glyphRun)
ConsoloniaPlatform.RaiseNotSupported(17, glyphRun);
throw new InvalidProgramException();
}

if (glyphRun.FontRenderingEmSize.IsNearlyEqual(0)) return;
if (!glyphRun.FontRenderingEmSize.IsNearlyEqual(1))
{
Expand All @@ -196,9 +195,9 @@ public void PushClip(Rect clip)

public void PushClip(RoundedRect clip)
{
if (clip.IsRounded)
if(clip.IsRounded)
ConsoloniaPlatform.RaiseNotSupported(2);

PushClip(clip.Rect);
}

Expand Down Expand Up @@ -352,7 +351,7 @@ void DrawPixelAndMoveHead(int count)
}
}
}

private void DrawStringInternal(IBrush foreground, string str, Point origin = new())
{
if (foreground is not FourBitColorBrush { Mode: PixelBackgroundMode.Colored } consoleColorBrush)
Expand Down
20 changes: 12 additions & 8 deletions src/Consolonia.Core/Drawing/FourBitColorBrush.cs
Original file line number Diff line number Diff line change
Expand Up @@ -43,19 +43,19 @@ public ConsoleColor Color
set => SetValue(ColorProperty, value);
}

public double Opacity => 1;
public ITransform Transform => null;
public RelativePoint TransformOrigin => RelativePoint.TopLeft;

// ReSharper disable once UnusedMember.Global used by Avalonia
// ReSharper disable once UnusedParameter.Global
public IBrush ProvideValue(IServiceProvider _)
{
return this;
}

public double Opacity => 1;
public ITransform Transform => null;
public RelativePoint TransformOrigin => RelativePoint.TopLeft;
}

public class FourBitBrushConverter : TypeConverter
public class FourBitBrushConverter : TypeConverter
{
public override bool CanConvertFrom(ITypeDescriptorContext context, Type sourceType)
{
Expand All @@ -66,11 +66,15 @@ public override object ConvertFrom(
ITypeDescriptorContext context, CultureInfo culture, object value)
{
if (value is string s)
return Enum.TryParse(s, out ConsoleColor result)
? result
{
return Enum.TryParse(s, out ConsoleColor result)
? result
: null;

}

return null;
}
}


}
16 changes: 8 additions & 8 deletions src/Consolonia.Core/Drawing/RenderTarget.cs
Original file line number Diff line number Diff line change
Expand Up @@ -67,13 +67,6 @@ void IDrawingContextLayerImpl.Blit(IDrawingContextImpl context)

bool IDrawingContextLayerImpl.CanBlit => true;

public IDrawingContextImpl CreateDrawingContext()
{
return new DrawingContextImpl(_consoleWindow, _bufferBuffer);
}

public bool IsCorrupted => false;


private void OnResized(Size size, WindowResizeReason reason)
{
Expand Down Expand Up @@ -117,7 +110,7 @@ private void RenderToDevice()
throw new InvalidOperationException("Caret is already shown");
caretPosition = new PixelBufferCoordinate(x, y);
}

/* todo: There is not IWindowImpl.Invalidate anymore.
if (!_consoleWindow.InvalidatedRects.Any(rect =>
rect.ContainsExclusive(new Point(x, y)))) continue;*/
Expand Down Expand Up @@ -228,5 +221,12 @@ public void Flush()
_stringBuilder.Clear();
}
}

public IDrawingContextImpl CreateDrawingContext()
{
return new DrawingContextImpl(_consoleWindow, _bufferBuffer);
}

public bool IsCorrupted => false;
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -10,11 +10,11 @@ namespace Consolonia.Core.Infrastructure
{
public class ArrowsAndKeyboardNavigationHandler : IKeyboardNavigationHandler
{
private readonly IKeyboardNavigationHandler _keyboardNavigationHandler;

//todo: check XTFocus https://github.com/jinek/Consolonia/issues/105#issuecomment-2089015880
private IInputRoot _owner;

private readonly IKeyboardNavigationHandler _keyboardNavigationHandler;

public ArrowsAndKeyboardNavigationHandler(IKeyboardNavigationHandler keyboardNavigationHandler)
{
_keyboardNavigationHandler = keyboardNavigationHandler;
Expand Down
26 changes: 14 additions & 12 deletions src/Consolonia.Core/Infrastructure/ConsoleWindow.cs
Original file line number Diff line number Diff line change
Expand Up @@ -31,8 +31,6 @@ public ConsoleWindow()
Console.FocusEvent += ConsoleOnFocusEvent;
}

private IMouseDevice MouseDevice { get; }

public void Dispose()
{
Closed?.Invoke();
Expand All @@ -42,7 +40,7 @@ public void Dispose()
Console.FocusEvent -= ConsoleOnFocusEvent;
Console.Dispose();
}

public void SetInputRoot(IInputRoot inputRoot)
{
_inputRoot = inputRoot;
Expand Down Expand Up @@ -78,7 +76,7 @@ public void SetFrameThemeVariant(PlatformThemeVariant themeVariant)
{
//todo:
}

public Size ClientSize
{
get
Expand Down Expand Up @@ -106,6 +104,7 @@ public Size ClientSize
public Compositor Compositor { get; } = new(null);
public Action Closed { get; set; }
public Action LostFocus { get; set; }
private IMouseDevice MouseDevice { get; }

public WindowTransparencyLevel TransparencyLevel => WindowTransparencyLevel.None;

Expand Down Expand Up @@ -241,14 +240,6 @@ public void SetExtendClientAreaTitleBarHeightHint(double titleBarHeight)
// ReSharper disable once UnassignedGetOnlyAutoProperty todo: what is this property
public Thickness OffScreenMargin { get; }

public object TryGetFeature(Type featureType)
{
if (featureType == typeof(ISystemNavigationManagerImpl))
return null;
if (featureType == typeof(ITextInputMethodImpl)) return null;
throw new NotImplementedException("Consider this");
}

private void ConsoleOnMouseEvent(RawPointerEventType type, Point point, Vector? wheelDelta,
RawInputModifiers modifiers)
{
Expand Down Expand Up @@ -342,5 +333,16 @@ await Dispatcher.UIThread.InvokeAsync(() =>
}, DispatcherPriority.Input);
}
}

public object TryGetFeature(Type featureType)
{
if (featureType == typeof(ISystemNavigationManagerImpl))
return null;
if (featureType == typeof(ITextInputMethodImpl))
{
return null;
}
throw new NotImplementedException("Consider this");
}
}
}
2 changes: 1 addition & 1 deletion src/Consolonia.Core/Infrastructure/ConsoloniaPlatform.cs
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ public void Initialize()
.Bind<IRenderTimer>().ToConstant(new UiThreadRenderTimer(120))
.Bind<IDispatcherImpl>().ToConstant(new ManagedDispatcherImpl(null))
/*.Bind<IRenderTimer>().ToConstant(new SleepLoopRenderTimer(120))*/
/*SleepLoopRenderTimer : IRenderTimer*/
/*SleepLoopRenderTimer : IRenderTimer*/
/*.Bind<IRenderLoop>().ToConstant(new RenderLoop()) todo: is internal now*/
.Bind<PlatformHotkeyConfiguration>().ToConstant(new PlatformHotkeyConfiguration(KeyModifiers.Control))
.Bind<IKeyboardDevice>().ToConstant(new ConsoleKeyboardDevice())
Expand Down
4 changes: 3 additions & 1 deletion src/Consolonia.Core/Infrastructure/ExceptionSink.cs
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,9 @@ public bool IsEnabled(LogEventLevel level, string area)
public void Log(LogEventLevel level, string area, object source, string messageTemplate)
{
Log(level, area, source, messageTemplate, Array.Empty<object>());
} // ReSharper disable UnusedMember.Global
}

// ReSharper disable UnusedMember.Global
public void Log<T0>(LogEventLevel level, string area, object source, string messageTemplate, T0 propertyValue0)
{
Log(level, area, source, messageTemplate, new object[] { propertyValue0 });
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -83,7 +83,9 @@ public void Print(PixelBufferCoordinate bufferPoint, ConsoleColor backgroundColo
if (str.Any(
c => ConsoleText.IsWideChar(c) &&
char.IsLetterOrDigit(c) /*todo: https://github.com/SlimeNull/NullLib.ConsoleEx/issues/2*/))
{
throw new NotSupportedException("Is not supposed to be rendered");
}

Console.Write(str);

Expand Down
3 changes: 1 addition & 2 deletions src/Consolonia.Core/InternalHelpers/CommonInternalHelper.cs
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,7 @@ namespace Consolonia.Core.InternalHelpers
internal static class CommonInternalHelper
{
public static bool IsNearlyEqual(this double value, double compareTo)
{
//todo: strange implementation for this name
{//todo: strange implementation for this name
return value.CompareTo(compareTo) == 0;
}

Expand Down
3 changes: 1 addition & 2 deletions src/Consolonia.Core/Styles/ResourceIncludeBase.cs
Original file line number Diff line number Diff line change
Expand Up @@ -46,12 +46,11 @@ public IStyle Loaded
return _loaded[0];
}
}

public bool TryGetResource(object key, ThemeVariant theme, out object value)
{
if (!_isLoading && Loaded is IResourceProvider p)
return p.TryGetResource(key, theme, out value);

value = null;
return false;
}
Expand Down
15 changes: 7 additions & 8 deletions src/Consolonia.Core/Text/FontManagerImpl.cs
Original file line number Diff line number Diff line change
Expand Up @@ -11,18 +11,22 @@ namespace Consolonia.Core.Text
/// </summary>
internal class FontManagerImpl : IFontManagerImpl
{
public static string GetTheOnlyFontFamilyName()
{
return "ConsoleDefault(F7D6533C-AC9D-4C4A-884F-7719A9B5DC0C)";
}

public string GetDefaultFontFamilyName()
{
return GetTheOnlyFontFamilyName();
}

string[] IFontManagerImpl.GetInstalledFontFamilyNames(bool checkForUpdates)
{
return new[] { GetTheOnlyFontFamilyName() };
return new []{GetTheOnlyFontFamilyName()};
}

public bool TryMatchCharacter(int codepoint, FontStyle fontStyle, FontWeight fontWeight,
FontStretch fontStretch,
public bool TryMatchCharacter(int codepoint, FontStyle fontStyle, FontWeight fontWeight, FontStretch fontStretch,
CultureInfo culture, out Typeface typeface)
{
throw new NotImplementedException();
Expand All @@ -40,10 +44,5 @@ public bool TryCreateGlyphTypeface(Stream stream, out IGlyphTypeface glyphTypefa
{
throw new NotImplementedException();
}

public static string GetTheOnlyFontFamilyName()
{
return "ConsoleDefault(F7D6533C-AC9D-4C4A-884F-7719A9B5DC0C)";
}
}
}
4 changes: 2 additions & 2 deletions src/Consolonia.Core/Text/GlyphRunImpl.cs
Original file line number Diff line number Diff line change
Expand Up @@ -20,10 +20,9 @@ public GlyphRunImpl(IGlyphTypeface glyphTypeface, IReadOnlyList<GlyphInfo> glyph
new Size(glyphInfos.Sum(info => info.GlyphAdvance), FontRenderingEmSize));
}

public ushort[] GlyphIndices { get; }

public void Dispose()
{

}

public IReadOnlyList<float> GetIntersections(float lowerLimit, float upperLimit)
Expand All @@ -36,5 +35,6 @@ public IReadOnlyList<float> GetIntersections(float lowerLimit, float upperLimit)
public double FontRenderingEmSize { get; }
public Point BaselineOrigin { get; }
public Rect Bounds { get; }
public ushort[] GlyphIndices { get; }
}
}
13 changes: 8 additions & 5 deletions src/Consolonia.Core/Text/TextShaper.cs
Original file line number Diff line number Diff line change
Expand Up @@ -12,11 +12,14 @@ public class TextShaper : ITextShaperImpl
public ShapedBuffer ShapeText(ReadOnlyMemory<char> text, TextShaperOptions options)
{
var glyphInfos = Convert(text.Span.ToString());

var shapedBuffer = new ShapedBuffer(text, glyphInfos.Length,
new GlyphTypeface(), 1, 0 /*todo: must be 1 for right to left?*/);

for (int i = 0; i < shapedBuffer.Length; i++) shapedBuffer[i] = glyphInfos[i];

for (int i = 0; i < shapedBuffer.Length; i++)
{
shapedBuffer[i] = glyphInfos[i];
}
return shapedBuffer;
}

Expand All @@ -27,8 +30,8 @@ public static GlyphInfo[] Convert(string str)

// ReSharper disable once InvertIf
if (str.Any(
c => ConsoleText.IsWideChar(c) &&
char.IsLetterOrDigit(c) /*todo: https://github.com/SlimeNull/NullLib.ConsoleEx/issues/2*/))
c => ConsoleText.IsWideChar(c) &&
char.IsLetterOrDigit(c) /*todo: https://github.com/SlimeNull/NullLib.ConsoleEx/issues/2*/))
{
StringBuilder stringBuilder = new();
foreach (char c in str)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
namespace Consolonia.Gallery.Gallery.GalleryViews
{
// ReSharper disable once UnusedType.Global
public class GalleryAnimatedLines : UserControl
public partial class GalleryAnimatedLines : UserControl
{
public GalleryAnimatedLines()
{
Expand Down
Loading

0 comments on commit 34c9336

Please sign in to comment.