Skip to content

http2: order declarations in core.js

Rodrigo Muino Tomonari requested to merge github/fork/Trott/ordering into master

Not sure how this will be received because maybe there's an ordering in there already that I'm not detecting, but I find the lengthy declarations at the start of core.js kind of arbitrary and hard to follow. This brings some ordering to it, and hopefully makes it easier to follow or at least not any more difficult to follow.

Order declarations:

  • public modules in alphabetical order
  • internal modules in alphabetical order
  • process.binding() calls in alphabetical order
  • exports in alphabetical order
Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines

Merge request reports