lib,src: make os.endianness() inlinable

bnoordhuis · bnoordhuis · commit 355b96b60559 · 2014-11-11T12:02:27.000+01:00
Turn os.endianness() from a run-time function into a pure JS function. Upsides: makes it a good candidate for inlining at the call site. Downsides: none that I can think of. PR-URL: node-forward/node#55 Reviewed-By: Chris Dickinson <christopher.s.dickinson@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Fedor Indutny <fedor@indutny.com>
diff --git a/lib/os.js b/lib/os.js
@@ -23,7 +23,6 @@ var binding = process.binding('os');
 var util = require('util');
 var isWindows = process.platform === 'win32';
 
-exports.endianness = binding.getEndianness;
 exports.hostname = binding.getHostname;
 exports.loadavg = binding.getLoadAvg;
 exports.uptime = binding.getUptime;
@@ -62,3 +61,8 @@ exports.getNetworkInterfaces = util.deprecate(function() {
 }, 'getNetworkInterfaces is now called `os.networkInterfaces`.');
 
 exports.EOL = isWindows ? '\r\n' : '\n';
+
+if (binding.isBigEndian)
+  exports.endianness = function() { return 'BE'; };
+else
+  exports.endianness = function() { return 'LE'; };
diff --git a/src/node_os.cc b/src/node_os.cc
@@ -48,6 +48,7 @@ namespace node {
 namespace os {
 
 using v8::Array;
+using v8::Boolean;
 using v8::Context;
 using v8::FunctionCallbackInfo;
 using v8::Handle;
@@ -59,13 +60,6 @@ using v8::String;
 using v8::Value;
 
 
-static void GetEndianness(const FunctionCallbackInfo<Value>& args) {
-  Environment* env = Environment::GetCurrent(args);
-  const char* rval = IsBigEndian() ? "BE" : "LE";
-  args.GetReturnValue().Set(OneByteString(env->isolate(), rval));
-}
-
-
 static void GetHostname(const FunctionCallbackInfo<Value>& args) {
   Environment* env = Environment::GetCurrent(args);
   char buf[MAXHOSTNAMELEN + 1];
@@ -300,7 +294,6 @@ void Initialize(Handle<Object> target,
                 Handle<Value> unused,
                 Handle<Context> context) {
   Environment* env = Environment::GetCurrent(context);
-  env->SetMethod(target, "getEndianness", GetEndianness);
   env->SetMethod(target, "getHostname", GetHostname);
   env->SetMethod(target, "getLoadAvg", GetLoadAvg);
   env->SetMethod(target, "getUptime", GetUptime);
@@ -310,6 +303,8 @@ void Initialize(Handle<Object> target,
   env->SetMethod(target, "getOSType", GetOSType);
   env->SetMethod(target, "getOSRelease", GetOSRelease);
   env->SetMethod(target, "getInterfaceAddresses", GetInterfaceAddresses);
+  target->Set(FIXED_ONE_BYTE_STRING(env->isolate(), "isBigEndian"),
+              Boolean::New(env->isolate(), IsBigEndian()));
 }
 
 }  // namespace os
