Skip to content

Conversation

@juj
Copy link
Collaborator

@juj juj commented Feb 6, 2019

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

@juj juj added the code size label Feb 6, 2019
Copy link
Member

@kripken kripken left a comment

Choose a reason for hiding this comment

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

Great!

@@ -1,38 +1,3 @@
/** @type {function(number, number=)} */
Copy link
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
Copy link
Collaborator Author

juj commented Feb 6, 2019

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