Skip to content

Commit

Permalink
remov deprecation messages in proxycommand
Browse files Browse the repository at this point in the history
  • Loading branch information
DeltaDizzy committed Mar 12, 2024
1 parent 7c2b556 commit 9575d41
Showing 1 changed file with 0 additions and 12 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -36,12 +36,6 @@ class ProxyCommand : public CommandHelper<Command, ProxyCommand> {
* DeferredCommand} instead.
*
* @param supplier the command supplier
* @deprecated This constructor's similarity to {@link DeferredCommand} is
* confusing and opens potential footguns for users who do not fully
* understand the semantics and implications of proxying, but who simply want
* runtime construction. Users who do know what they are doing and need a
* supplier-constructed proxied command should instead proxy a DeferredCommand
* using the <code>AsProxy</code> decorator.
* @see DeferredCommand
*/
explicit ProxyCommand(wpi::unique_function<Command*()> supplier);
Expand All @@ -55,12 +49,6 @@ class ProxyCommand : public CommandHelper<Command, ProxyCommand> {
* DeferredCommand} instead.
*
* @param supplier the command supplier
* @deprecated This constructor's similarity to {@link DeferredCommand} is
* confusing and opens potential footguns for users who do not fully
* understand the semantics and implications of proxying, but who simply want
* runtime construction. Users who do know what they are doing and need a
* supplier-constructed proxied command should instead proxy a DeferredCommand
* using the <code>AsProxy</code> decorator.
* @see DeferredCommand
*/
explicit ProxyCommand(wpi::unique_function<CommandPtr()> supplier);
Expand Down

0 comments on commit 9575d41

Please sign in to comment.