Skip to content

Remove deprecated function Pointer_stringify() - #8011

Merged
juj merged 1 commit into
emscripten-core:incomingfrom
juj:remove_pointer_stringify
Feb 6, 2019
Merged

juj merged 1 commit into
emscripten-core:incomingfrom
juj:remove_pointer_stringify

Conversation

@juj

@juj juj commented Feb 6, 2019

Copy link
Copy Markdown
Collaborator

Removes the last references to Pointer_stringify() from the codebase.

@juj juj added the code size label Feb 6, 2019

@kripken kripken left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Great!

Comment thread src/runtime_strings.js
@@ -1,38 +1,3 @@
/** @type {function(number, number=)} */

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

It may be worth replacing this with

#if ASSERTIONS
function Pointer_stringify() {
  abort("this function has been removed - you should use [..]");
}
#endif

which would help debug builds direct people to what they need to do.

@juj
juj force-pushed the remove_pointer_stringify branch from 58a3e6a to 838f297 Compare February 6, 2019 19:42
@juj

juj commented Feb 6, 2019

Copy link
Copy Markdown
Collaborator Author

Ok, added it back with an error message.

@juj
juj merged commit 985ce63 into emscripten-core:incoming Feb 6, 2019
gittup added a commit to gittup/Effekseer that referenced this pull request May 31, 2021
Pointer_stringify() was deprecated in emscripten
(emscripten-core/emscripten#8011)
durswd pushed a commit to effekseer/Effekseer that referenced this pull request Jun 1, 2021
durswd pushed a commit to effekseer/Effekseer that referenced this pull request Jun 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants