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.

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

new_s and new_s_i opcodes #330

Closed
Whiteknight opened this issue May 14, 2011 · 3 comments
Closed

new_s and new_s_i opcodes #330

Whiteknight opened this issue May 14, 2011 · 3 comments

Comments

@Whiteknight
Copy link
Contributor

We have two opcodes for creating new, empty strings: new_s and new_s_i (both in src/ops/string.ops). There is a note in that file:

XXX: Do these ops make sense with immutable strings?

My answer is "no".

We should deprecate and remove these ops because they are worthless now.

Originally http://trac.parrot.org/parrot/ticket/2111

@jkeenan
Copy link
Contributor

jkeenan commented May 16, 2011

Replying to whiteknight:

We have two opcodes for creating new, empty strings: new_s and new_s_i (both in src/ops/string.ops). There is a note in that file:

We should deprecate and remove these ops because they are worthless now.

Given the problem we've had with deprecations in recent days, I feel we should ask:

Which projects built on top of Parrot use these opcodes?

If there are such usages, what is our migration plan?

Thank you very much.

kid51

@bacek
Copy link
Member

bacek commented Nov 20, 2011

Hello.

 https://github.com/parrot/parrot/issues/203

new_s_i is deprecated.

new_s should stay to avoid null pointer exception little bit more easily.

-- Bacek

@ghost ghost assigned Whiteknight May 9, 2012
@Whiteknight
Copy link
Contributor Author

new_s_i does not appear to exist anymore. new_s, as requested by bacek, can stay. I'm closing this ticket now.

@Whiteknight Whiteknight removed their assignment Mar 7, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants