Implement Purple.CommandManager

Review Request #3619 — Created Oct. 30, 2024 and submitted

Information

pidgin/pidgin
default

Reviewers

This ended up deviating from the design a bit, but I think I got everything we
need at least for now.

Ran the tests under valgrind and called in the turtles for the rest. Also ran in a devenv to make sure everything started fine.

Summary ID
Implement Purple.CommandManager
This ended up deviating from the design a bit, but I think I got everything we need at least for now.
ee14efdddeac58f064cb5379ae53f487c8b940ed
Description From Last Updated

I debated on making the command transfer full. Any thoughts?

grimgrim

Could throw in an "is" here, I think.

ivanhoeivanhoe

Missing "add"?

ivanhoeivanhoe

One "different" seems enough but I think there could be a "with" after the "but".

ivanhoeivanhoe
grim
  1. 
      
  2. libpurple/purplecommandmanager.h (Diff revision 1)
     
     
    Show all issues

    I debated on making the command transfer full. Any thoughts?

  3. 
      
grim
ivanhoe
  1. Ship It!
  2. libpurple/purplecommand.h (Diff revision 2)
     
     
    Show all issues

    Could throw in an "is" here, I think.

  3. libpurple/tests/test_command_manager.c (Diff revision 2)
     
     
    Show all issues

    Missing "add"?

  4. libpurple/tests/test_command_manager.c (Diff revision 2)
     
     
    Show all issues

    One "different" seems enough but I think there could be a "with" after the "but".

  5. 
      
grim
grim
Review request changed
Status:
Completed