Günther Foidl

Benchmarks for .NET Full with issues

2

E.g. from run https://dev.azure.com/gh-gfoidl/github-Projects/_build/results?buildId=902&view=results ``` ini BenchmarkDotNet=v0.12.0, OS=Windows 10.0.17763.1039 (1809/October2018Update/Redstone5), VM=Hyper-V Intel Xeon CPU E5-2673 v3 2.40GHz, 1 CPU, 2 logical and 2 physical cores [Host] : .NET Framework 4.8...

https://github.com/dotnet/runtime/issues/11524 got fixed, so `Unsafe.WriteUnaligned` can be used (for correctness). E.g. https://github.com/gfoidl/Base64/blob/90470c28bdd511de23e6d5f3a901b42346e7af17/source/gfoidl.Base64/Internal/Vector256Helper.cs#L70-L75 Search for: https://github.com/dotnet/coreclr/issues/21132

For context see https://github.com/gfoidl/Base64/pull/124#issuecomment-601486041 Due to JIT limits (too many locals) the avx-codepath won't inline. Thus the change got [reverted] (https://github.com/gfoidl/Base64/pull/124#issuecomment-601877657). Recheck later if JIT improved, so this method can...

Typings: SerializerOptions.indexes allows `boolean | null | undefined`

1

SerializerOptions.indexes is typed as `indexes?: boolean;` but according the docs (ReadMe), PR descriptions and tests (https://github.com/axios/axios/blob/bdf493cf8b84eb3e3440e72d5725ba0f138e0451/test/unit/adapters/http.js#L1505-L1506) `null` is also a valid value.

Remove some branches in HtmlHelper.TryParseHtmlTagOpenTag by using bitmask

1

Eliminate some branches in `HtmlHelper.TryParseHtmlTagOpenTag` (due to `||` comparisons) by using a bitmask approach instead. Further the generated machine code has a `test jump`-combo, which allows optimizations at cpu-level (macro...

Vector{128,256} operations that use MmShuffle fall back to method call

6

### Prerequisites - [X] I have written a descriptive issue title - [X] I have verified that I am running the latest version of ImageSharp - [X] I have verified...

In the form control wrapper wrap the elements inside a <div>

4

Without wrapping inside a `` the valiation feedback may be shown on other input-controls too, which is not intended. Thus we need to wrap all controls related to the input...

CLink emit click's argument contains the mouse events

6

W/o providing the (native) mouse event no consumer can react accordingly. With this change the mouse event is emitted as first argument in the click event. https://github.com/coreui/coreui-vue/blob/a5c0ad536950d30cac0ef24fd4982b44a9a1df57/packages/coreui-vue/src/components/link/CLink.ts#L44-L46

Utf8Literals instead hand-crafted byte-sequences

1

Additional: * remove `movsxd`, as they're not needed * used C# language to denote if an argument should be ignored

Vector128.Min<int> produces sub-optimal code on x64

5

Repro: ```c# [MethodImpl(MethodImplOptions.NoInlining)] static Vector128 Foo(Vector128 a, Vector128 b) { return Vector128.Min(a, b); } ``` Actual: ```asm G_M000_IG01: ;; offset=0000H C5F877 vzeroupper G_M000_IG02: ;; offset=0003H C5F91002 vmovupd xmm0, xmmword ptr...