chipsalliance/chisel

RFC: Deprecate either Decoupled or DecoupledIO?

Aberta

#2.463 aberto em 28 de mar. de 2022

 (12 comentários) (1 reação) (0 responsável)Scala (651 forks)batch import
good first issue

Métricas do repositório

Stars
 (4.658 estrelas)
Métricas de merge de PR
 (Mesclagem média 3d 21h) (16 fundiu PRs em 30d)

Description

Type of issue: feature request

Impact: API modification

Development Phase: proposal

Other information

If the current behavior is a bug, please provide the steps to reproduce the problem:

What is the current behavior?

User does something like:

val foo: DecoupledIO[Bar] = Decoupled(bar)

There is a mental burden to remember which one is DecoupledIO and which one is Decoupled, for no good reason.

What is the expected behavior?

User would write:

val foo: Decoupled[Bar] = Decoupled(bar)

OR

val foo: DecoupledIO[Bar] = DecoupledIO(bar)

We'd deprecate the other one for a version or so.

Similar for ReadyValids and such, I didn't do a comprehensive survey

Please tell us about your environment:

What is the use case for changing the behavior?

Less mental load for users.

Guia do colaborador