Ticket #190 (closed todo: fixed)

Opened 13 years ago

Last modified 13 years ago

deprecated: ref pmc

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

Description


Change History

  Changed 13 years ago by kjs

  • platform set to all
  • type changed from bug to todo

follow-up: ↓ 3   Changed 13 years ago by jkeenan

  • status changed from new to assigned
  • owner set to jkeenan

Have begun working on this in the deprecate_ref branch in SVN.

in reply to: ↑ 2   Changed 13 years ago by jkeenan

  • status changed from assigned to new
  • owner jkeenan deleted

Replying to jkeenan:

Have begun working on this in the deprecate_ref branch in SVN.

Woah! Realized this is more than just removing a few files. Above my pay grade. Removing that branch. Someone else will have to pick this up.

  Changed 13 years ago by allison

  • milestone changed from 1.0 to 1.1

  Changed 13 years ago by bacek

Hello.

I have a question about Ref. It's deprecated, but used internally by "Undef" and "scalar". Should we deprecate "scalar" as well and merge semantic of Ref into Undef?

-- Bacek

  Changed 13 years ago by cotto

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

Ref and SharedRef are removed in r38030. I assumed that SharedRef was implicitly covered by Ref's deprecation notice, since the one extends the other. I made a best effort at updating the code in undef and scalar that used Ref in r38029. Unfortunately the code changed by that revision doesn't have any test coverage, so I'm not sure if it was the right approach.

Note: See TracTickets for help on using tickets.