delta/node-new.git/benchmark/dgram/offset-length.js, branch try_Python3_on_macOS

benchmark: refactor for consistent style

2019-02-07T06:18:31+00:00

Code in benchmark directory sometimes uses `function () {}` for
anonymous callbacks and sometimes uses `() => {}`. Multi-line arrays
sometimes have a trailing comma and sometimes do not. Update to always
use arrow functions for anonymous callbacks and trailing commas for
multiline arrays.

PR-URL: https://github.com/nodejs/node/pull/25944
Reviewed-By: Anto Aravinth 
Reviewed-By: James M Snell

tools: capitalize sentences

2018-12-10T16:07:18+00:00

This adds the `capitalized-comments` eslint rule to verify that
actual sentences use capital letters as starting letters. It ignores
special words and all lines below 62 characters.

PR-URL: https://github.com/nodejs/node/pull/24808
Reviewed-By: Sam Ruby 
Reviewed-By: Trivikram Kamat 
Reviewed-By: Ujjwal Sharma 
Reviewed-By: Sakthipriyan Vairamani

benchmark: (dgram) use destructuring

2018-01-23T00:29:33+00:00

PR-URL: https://github.com/nodejs/node/pull/18250
Reviewed-By: Matteo Collina 
Reviewed-By: James M Snell

benchmark: var to const

2017-09-20T00:14:38+00:00

PR-URL: https://github.com/nodejs/node/pull/13757
Reviewed-By: Matteo Collina 
Reviewed-By: James M Snell 
Reviewed-By: Luigi Pinca 
Reviewed-By: Gibson Fahnestock 
Reviewed-By: Tobias Nießen

benchmark: removed unused arguments from callbacks

2017-08-21T21:57:44+00:00

Removed unused arguments 'buf' and 'rinfo' from the callbacks.

PR-URL: https://github.com/nodejs/node/pull/14919
Reviewed-By: Rich Trott 
Reviewed-By: James M Snell 
Reviewed-By: Tobias Nießen 
Reviewed-By: Luigi Pinca 
Reviewed-By: Ruben Bridgewater

benchmark,lib,test: use braces for multiline block

2017-06-23T21:43:20+00:00

For if/else and loops where the bodies span more than one line, use
curly braces.

PR-URL: https://github.com/nodejs/node/pull/13828
Ref: https://github.com/nodejs/node/pull/13623#discussion_r123048602
Reviewed-By: Anna Henningsen 
Reviewed-By: Colin Ihrig 
Reviewed-By: Michaël Zasso 
Reviewed-By: James M Snell

benchmark: favor === over ==

2016-08-09T21:40:01+00:00

Refs: https://github.com/nodejs/node/pull/7961#discussion_r73788501
PR-URL: https://github.com/nodejs/node/pull/8000
Reviewed-By: Brian White 
Reviewed-By: Andreas Madsen 
Reviewed-By: Anna Henningsen 
Reviewed-By: Colin Ihrig 
Reviewed-By: James M Snell 
Reviewed-By: Johan Bergström

benchmark: missing process.exit after bench.end

2016-07-26T11:21:50+00:00

Previously bench.end would call process.exit(0) however this is rather
confusing and indeed a few benchmarks had code that assumed otherwise.

This adds process.exit(0) to the benchmarks that needs it.

PR-URL: https://github.com/nodejs/node/pull/7094
Reviewed-By: Trevor Norris 
Reviewed-By: Jeremiah Senkpiel 
Reviewed-By: Brian White 
Reviewed-By: Anna Henningsen

buffer: add .from(), .alloc() and .allocUnsafe()

2016-03-16T15:34:02+00:00

Several changes:

* Soft-Deprecate Buffer() constructors
* Add `Buffer.from()`, `Buffer.alloc()`, and `Buffer.allocUnsafe()`
* Add `--zero-fill-buffers` command line option
* Add byteOffset and length to `new Buffer(arrayBuffer)` constructor
* buffer.fill('') previously had no effect, now zero-fills
* Update the docs

PR-URL: https://github.com/nodejs/node/pull/4682
Reviewed-By: Сковорода Никита Андреевич 
Reviewed-By: Stephen Belanger

tools,benchmark: increase lint compliance

2016-02-28T04:15:17+00:00

In the hopes of soon having the benchmark code linted, this change
groups all the likely non-controversial lint-compliance changes such as
indentation, semi-colon usage, and single-vs.-double quotation marks.

Other lint rules may have subtle performance implications in the V8
currently shipped with Node.js. Those changes will require more careful
review and will be in a separate change.

PR-URL: https://github.com/nodejs/node/pull/5429
Reviewed-By: Roman Reiss 
Reviewed-By: Brian White