Skip to content
This repository was archived by the owner on Oct 15, 2020. It is now read-only.

Conversation

@kfarnung
Copy link
Contributor

@kfarnung kfarnung commented Dec 8, 2017

  • Replaced usages of NULL with nullptr
  • Fixed indentation in class
Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines
Affected core subsystem(s)

chakrashim

@kfarnung kfarnung self-assigned this Dec 8, 2017
@kfarnung kfarnung requested a review from boingoing December 8, 2017 23:17
@kfarnung
Copy link
Contributor Author

kfarnung commented Dec 8, 2017

Copy link
Contributor

@boingoing boingoing left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

* Replaced usages of NULL with nullptr
* Fixed indentation in class

PR-URL: nodejs#441
Reviewed-By: Jimmy Thomson <jithomso@microsoft.com>
Reviewed-By: Taylor Woll <taylor.woll@microsoft.com>
PR-URL: nodejs#441
Reviewed-By: Jimmy Thomson <jithomso@microsoft.com>
Reviewed-By: Taylor Woll <taylor.woll@microsoft.com>
@kfarnung kfarnung merged commit ffa70ff into nodejs:master Dec 9, 2017
@kfarnung kfarnung deleted the lintfix branch December 9, 2017 02:32
kfarnung added a commit to kfarnung/node-chakracore that referenced this pull request Jan 10, 2018
* Replaced usages of NULL with nullptr
* Fixed indentation in class

PR-URL: nodejs#441
Reviewed-By: Jimmy Thomson <jithomso@microsoft.com>
Reviewed-By: Taylor Woll <taylor.woll@microsoft.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants