Skip to content

Commit

Permalink
Stop abusing enums in Image/ExifUtil.ts
Browse files Browse the repository at this point in the history
  • Loading branch information
TimWolla committed Aug 2, 2023
1 parent 6b64df9 commit ec58d20
Show file tree
Hide file tree
Showing 2 changed files with 44 additions and 27 deletions.
41 changes: 20 additions & 21 deletions ts/WoltLabSuite/Core/Image/ExifUtil.ts
Original file line number Diff line number Diff line change
Expand Up @@ -7,27 +7,26 @@
* @module WoltLabSuite/Core/Image/ExifUtil
* @woltlabExcludeBundle tiny
*/
/* eslint-disable @typescript-eslint/no-unsafe-enum-comparison */

const enum Tag {
SOI = 0xd8, // Start of image
APP0 = 0xe0, // JFIF tag
APP1 = 0xe1, // EXIF / XMP
APP2 = 0xe2, // General purpose tag
APP3 = 0xe3, // General purpose tag
APP4 = 0xe4, // General purpose tag
APP5 = 0xe5, // General purpose tag
APP6 = 0xe6, // General purpose tag
APP7 = 0xe7, // General purpose tag
APP8 = 0xe8, // General purpose tag
APP9 = 0xe9, // General purpose tag
APP10 = 0xea, // General purpose tag
APP11 = 0xeb, // General purpose tag
APP12 = 0xec, // General purpose tag
APP13 = 0xed, // General purpose tag
APP14 = 0xee, // Often used to store copyright information
COM = 0xfe, // Comments
}

const Tag = {
SOI: 0xd8, // Start of image
APP0: 0xe0, // JFIF tag
APP1: 0xe1, // EXIF / XMP
APP2: 0xe2, // General purpose tag
APP3: 0xe3, // General purpose tag
APP4: 0xe4, // General purpose tag
APP5: 0xe5, // General purpose tag
APP6: 0xe6, // General purpose tag
APP7: 0xe7, // General purpose tag
APP8: 0xe8, // General purpose tag
APP9: 0xe9, // General purpose tag
APP10: 0xea, // General purpose tag
APP11: 0xeb, // General purpose tag
APP12: 0xec, // General purpose tag
APP13: 0xed, // General purpose tag
APP14: 0xee, // Often used to store copyright information
COM: 0xfe, // Comments
};

// Known sequence signatures
const _signatureEXIF = "Exif";
Expand Down
30 changes: 24 additions & 6 deletions wcfsetup/install/files/js/WoltLabSuite/Core/Image/ExifUtil.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

0 comments on commit ec58d20

Please sign in to comment.