atom / node-spellchecker

SpellChecker Node Module
http://atom.github.io/node-spellchecker
MIT License
300 stars 108 forks source link

Use 'Nan::Set' #135

Open abetomo opened 4 years ago

abetomo commented 4 years ago

Requirements for Contributing a Bug Fix

Identify the Bug

https://github.com/atom/node-spellchecker/issues/134

Description of the Change

Since obj->Set is an error, we use Nan::Set.

(#120 also seems to have the same fix in it.)

Alternate Designs

130 is a similar fix.

I thought it would be better to use Nan::Set.

Possible Drawbacks

Verification Process

I compiled and ran the unit tests.

Release Notes