#266: Packfile API cleanup: rename Parrot_readbc to Parrot_pbc_read, _loadbc =>
_pbc_load
--------------------+-------------------------------------------------------
Reporter: rurban | Owner: jkeenan
Type: todo | Status: new
Priority: minor | Milestone:
Component: core | Version:
Severity: low | Resolution:
Keywords: | Lang:
Patch: new | Platform:
--------------------+-------------------------------------------------------
Changes (by jkeenan):
* owner: rurban => jkeenan
* priority: normal => minor
* status: reopened => new
* severity: medium => low
Comment:
I was looking at ''DEPRECATED.pod'' tonight and noticed that it still
contains this notice:
{{{
Parrot_readbc, Parrot_loadbc. [eligible in 1.1]
They have been renamed to "Parrot_pbc_read", "Parrot_pbc_load".
"Parrot_pbc_read" has one third int argument.
<https://trac.parrot.org/parrot/ticket/266>
}}}
I grepped the repository and found that the only usages of `Parrot_readbc`
and `Parrot_loadbc` were in this part of ''include/parrot/embed.h'':
{{{
87 /* Deprecated */
88
89 /* with 0.9.1 TT #266 */
90 #define Parrot_readbc(i, c) Parrot_pbc_read((i), (c), 0)
91 #define Parrot_loadbc(i, pf) Parrot_pbc_load((i), (pf))
92
}}}
... which was added by rurban as part of r36680.
Now, since we are past 1.1, I believe that we should be able to remove
this code as well, correct? If so, I will do so and then remove the
notice from ''DEPRECATED.pod''.
Thank you very much.
kid51
--
Ticket URL: <https://trac.parrot.org/parrot/ticket/266#comment:9>
Parrot <https://trac.parrot.org/parrot/>
Parrot Development
_______________________________________________
parrot-tickets mailing list
[email protected]
http://lists.parrot.org/mailman/listinfo/parrot-tickets
