Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Finalizers take function as first argument (closes #16307) #24605

Merged
merged 8 commits into from
Nov 21, 2017
Merged

Finalizers take function as first argument (closes #16307) #24605

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
b0bf9c2
=make finalizers take function as first argument
oxinabox Nov 14, 2017
747444b
=update all files so finalizers take functions as their first argument
oxinabox Nov 14, 2017
1534c38
=remove pointless type constraint
oxinabox Nov 15, 2017
687e86a
=add news for #24605
oxinabox Nov 15, 2017
1c3b17a
=spaces not tabs
oxinabox Nov 16, 2017
03ff4fb
=fix whitespacing and phrasing
oxinabox Nov 21, 2017
f94d5c9
=writing news
oxinabox Nov 21, 2017
9516d83
=writing news
oxinabox Nov 21, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
=writing news
  • Loading branch information
@oxinabox
oxinabox committed Nov 21, 2017
commit f94d5c97b371c61cab79736767eede23468d85b9
4 changes: 2 additions & 2 deletions NEWS.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -271,8 +271,8 @@ This section lists changes that do not have deprecation warnings.

* `finalizer` now takes functions or pointers as its first argument, and the object being
finalized as its second (rather than the reverse). For the majorities of use cases
Copy link
Member

Choose a reason for hiding this comment

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

"majorities" -> "majority"?

deprecation warnings will be triggered. However, deprecation warning will not trigger where:
(1) the callable argument is not a subtype of `Function`; or (2) when both arguments are
deprecation warnings will be triggered. However, deprecation warnings will not trigger where
(1) the callable argument is not a subtype of `Function`; or (2) both arguments are
`Function`s or `Ptr{Void}`s ([#24605]).

Library improvements
Expand Down