Ticket #107 (closed RFC: fixed)

Opened 6 years ago

Last modified 6 years ago

Deprecated: Data::Escape

Reported by: coke Owned by:
Priority: normal Milestone:
Component: library Version:
Severity: medium Keywords:
Cc: Language:
Patch status: Platform:

Description

I propose we remove runtime/parrot/library/Data/Escape.pir ; it provides a single method for escaping strings, which we now have in the form of an opcode, "escape".

Minor gotchas:

* The library provides for the ability to escape " and ' differently; not sure this is needed (Though it is tested for.)

* The library and the opcode generate different escaped representations: e.g. opcode does "\u0777", library does "\x{777}"

Change History

Changed 6 years ago by coke

  • component changed from none to library

Changed 6 years ago by coke

  • summary changed from remove Data::Escape to Deprecated: Data::Escape

Changed 6 years ago by jkeenan

Can someone supply a patch which resolves the 'gotchas'? If so, then we can decide whether to proceed with the deprecation or not.

Thank you very much.
kid51

Changed 6 years ago by coke

I haven't heard anyone say they need that functionality.

The only thing stopping me from ripping it out at this point is the Streams test that fails when I try.

Changed 6 years ago by coke

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

The streams tests were fixed in r36266; library removed in r36861.

languages/dotnet will need to update to track this change.

Note: See TracTickets for help on using tickets.