Ticket #780 (closed RFC: wontfix)

Opened 5 years ago

Last modified 3 years ago

[RFC] deprecated parrot_new_pointer_hash and parrot_new_cstring_hash

Reported by: bacek Owned by:
Priority: normal Milestone:
Component: core Version: 1.3.0
Severity: medium Keywords:
Cc: Language:
Patch status: Platform:

Description

Hello.

Currently parrot's hash (not PMC Hash) officially supports C-strings and C-pointers as keys. I can't find any usage of it and propose to deprecate them. (And make C-string and C-pointer keys in hash it virtually impossible to make bullet-proof anyway)

-- Bacek

Change History

Changed 5 years ago by allison

Parrot's Hash PMC is really just a thin wrapper around Hash * (that is to say, they're the same thing).

Both functions are used internally, one to create a storage location for thread data, and the other to create a storage location for the interpreter's string encoding data.

So, agreed in general principle, but deprecation is a large task. Rate this RFC as "long-term possible".

Changed 5 years ago by jkeenan

  • component changed from none to core

Changed 3 years ago by jkeenan

Can we get an update on the status of this ticket?

Thank you very much.

kid51

Changed 3 years ago by bacek

  • status changed from new to closed
  • resolution set to wontfix

Resolve ticket as wontfix. cstring hash is used for constant strings. cpointers are less harmful due very limited usage.

Note: See TracTickets for help on using tickets.