Skip to content

Commit

Permalink
doc: unambiguously mark deprecated signatures
Browse files Browse the repository at this point in the history
Currently, deprecation notices are always right between two function
signatures and it's virtually impossible to be certain whether they
refer to the previous signature or the next signature.

PR-URL: nodejs/node-addon-api#942
Reviewed-By: Nicola Del Gobbo <[email protected]>
Reviewed-By: Gabriel Schulhof <[email protected]>
Reviewed-By: Chengzhong Wu <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
  • Loading branch information
Marlyfleitas committed Mar 29, 2021
1 parent 1f71749 commit ece1572
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions doc/property_descriptor.md
Original file line number Diff line number Diff line change
Expand Up @@ -138,7 +138,7 @@ The name of the property can be any of the following types:
- `napi_value value`
- `Napi::Name`

**This signature is deprecated. It will result in a memory leak if used.**
**The above signature is deprecated. It will result in a memory leak if used.**

```cpp
static Napi::PropertyDescriptor Napi::PropertyDescriptor::Accessor (
Expand Down Expand Up @@ -186,7 +186,7 @@ The name of the property can be any of the following types:
- `napi_value value`
- `Napi::Name`

**This signature is deprecated. It will result in a memory leak if used.**
**The above signature is deprecated. It will result in a memory leak if used.**

```cpp
static Napi::PropertyDescriptor Napi::PropertyDescriptor::Accessor (
Expand Down Expand Up @@ -236,7 +236,7 @@ The name of the property can be any of the following types:
- `napi_value value`
- `Napi::Name`

**This signature is deprecated. It will result in a memory leak if used.**
**The above signature is deprecated. It will result in a memory leak if used.**

```cpp
static Napi::PropertyDescriptor Napi::PropertyDescriptor::Function (
Expand Down

0 comments on commit ece1572

Please sign in to comment.