From 069653d61e218450ae285ec21e0e1e5d4c33b526 Mon Sep 17 00:00:00 2001 From: Jason Nelson Date: Mon, 11 Jun 2018 12:39:45 -0700 Subject: [PATCH] Revert "Eliminate allocation processing the identifier in ProcessApp2Marker" This reverts commit e4bddc786f044eb60aacff5fb88bd292ade677e4. --- .../Formats/Jpeg/PdfJsPort/PdfJsJpegDecoderCore.cs | 14 +++++--------- 1 file changed, 5 insertions(+), 9 deletions(-) diff --git a/src/ImageSharp/Formats/Jpeg/PdfJsPort/PdfJsJpegDecoderCore.cs b/src/ImageSharp/Formats/Jpeg/PdfJsPort/PdfJsJpegDecoderCore.cs index cd893900e3..752e72dd2e 100644 --- a/src/ImageSharp/Formats/Jpeg/PdfJsPort/PdfJsJpegDecoderCore.cs +++ b/src/ImageSharp/Formats/Jpeg/PdfJsPort/PdfJsJpegDecoderCore.cs @@ -482,20 +482,16 @@ namespace SixLabors.ImageSharp.Formats.Jpeg.PdfJsPort private void ProcessApp2Marker(int remaining) { // Length is 14 though we only need to check 12. - const int IccLength = 14; - if (remaining < IccLength || this.IgnoreMetadata) + const int Icclength = 14; + if (remaining < Icclength || this.IgnoreMetadata) { this.InputStream.Skip(remaining); return; } -#if NETCOREAPP2_1 - byte[] identifier = new byte[IccLength]; // 14 bytes -#else - Span identifer = stackalloc byte[IccLength]; -#endif - this.InputStream.Read(identifier, 0, IccLength); - remaining -= IccLength; // We have read it by this point + byte[] identifier = new byte[Icclength]; + this.InputStream.Read(identifier, 0, Icclength); + remaining -= Icclength; // We have read it by this point if (ProfileResolver.IsProfile(identifier, ProfileResolver.IccMarker)) {