1636424 - Use standard/MFBT functions in DataView

Status: RESOLVED FIXED

(Core :: JavaScript: Standard Library, enhancement)

Description

[André Bargull [:anba]]

• DataToRepType can be replaced with mozilla::UnsignedStdintTypeForSize
• MOZ_ALIGNOF can be replaced with alignof
• needToSwapBytes can be replaced with mozilla::NativeEndian

Comment 1

[André Bargull [:anba]]

Attachment: Bug 1636424 - Part 1: Use UnsignedStdintTypeForSize to determine the unsigned integer representation. r=jandem!

Comment 2

[André Bargull [:anba]]

Attachment: Bug 1636424 - Part 2: Replace MOZ_ALIGNOF with alignof. r=jandem!

Depends on D74394

Comment 3

[André Bargull [:anba]]

Attachment: Bug 1636424 - Part 3: Use mozilla::NativeEndian to swap bytes in DataViews. r=jandem!

Depends on D74395

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/fcf44397b223
Part 1: Use UnsignedStdintTypeForSize to determine the unsigned integer representation. r=jandem
https://hg.mozilla.org/integration/autoland/rev/be7550014c77
Part 2: Replace MOZ_ALIGNOF with alignof. r=jandem
https://hg.mozilla.org/integration/autoland/rev/d50ab362c108
Part 3: Use mozilla::NativeEndian to swap bytes in DataViews. r=jandem

Comment 5

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/fcf44397b223
https://hg.mozilla.org/mozilla-central/rev/be7550014c77
https://hg.mozilla.org/mozilla-central/rev/d50ab362c108