Ticket #190 (closed todo: fixed)

Opened 4 years ago

Last modified 4 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 4 years ago by kjs

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

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

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

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

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

  • owner jkeenan deleted
  • status changed from assigned to new

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 4 years ago by allison

  • milestone changed from 1.0 to 1.1

  Changed 4 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 4 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.