Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(125)

Issue 10205009: [OTS] TAG() macro for big endian (Closed)

Created:
8 years, 8 months ago by bashi
Modified:
8 years, 7 months ago
Reviewers:
jfkthame, Yusuke Sato, agl
CC:
chromium-reviews, agl
Visibility:
Public.

Description

[OTS] TAG() macro for big endian Add TAG() macro for big endian. BUG=chromium:124629 TEST=ran test_{un,}malicious_fonts.sh

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+5 lines, -1 line) Patch
M src/ots.cc View 1 chunk +5 lines, -1 line 1 comment Download

Messages

Total messages: 4 (0 generated)
bashi
Hi Yusuke-san, Could you take a look?
8 years, 8 months ago (2012-04-24 08:44:24 UTC) #1
agl
lgtm
8 years, 8 months ago (2012-04-24 14:32:49 UTC) #2
Yusuke Sato
LGTM
8 years, 8 months ago (2012-04-24 17:44:44 UTC) #3
jfkthame
8 years, 7 months ago (2012-04-28 09:10:46 UTC) #4
https://chromiumcodereview.appspot.com/10205009/diff/1/src/ots.cc
File src/ots.cc (right):

https://chromiumcodereview.appspot.com/10205009/diff/1/src/ots.cc#newcode86
src/ots.cc:86: #if defined(__BIG_ENDIAN__) || defined(_BIG_ENDIAN)
Unfortunately, this doesn't work as intended on some systems - in the Android
NDK, and (at least some) OpenBSD systems, <endian.h> defines the symbol
_BIG_ENDIAN (with various levels of underscore-decoration) not to mean "this
system is big-endian" but as a magic constant to compare with _BYTE_ORDER. As a
result, the big-endian option here is chosen even on little-endian
architectures.

(See mozilla issue https://bugzilla.mozilla.org/show_bug.cgi?id=747816#c10.)

Powered by Google App Engine
This is Rietveld 408576698