Fix the docs and do some minor clean up for PurpleQueuedOutputStream

Review Request #1908 — Created Oct. 8, 2022 and submitted

Information

pidgin/pidgin
default

Reviewers

Fix the docs and do some minor clean up for PurpleQueuedOutputStream

Built the docs and verified all of symbols were now documented.

Summary ID
Fix the docs and do some minor clean up for PurpleQueuedOutputStream
df2626e11b5779a2ad980c605d5325312759b75f
Description From Last Updated

Argument name doesn't match.

QuLogicQuLogic

The other error doesn't have (optional); should be consistent.

QuLogicQuLogic

Space before =.

QuLogicQuLogic

This doesn't match the argument name. You should have got a warning about that.

QuLogicQuLogic
QuLogic
  1. 
      
  2. libpurple/queuedoutputstream.h (Diff revision 1)
     
     
    Show all issues

    Argument name doesn't match.

  3. libpurple/queuedoutputstream.h (Diff revision 1)
     
     
    Show all issues

    The other error doesn't have (optional); should be consistent.

    1. uh which other error?

    2. Ah, I'm not sure, I think I confused it with the cancellable.

  4. libpurple/queuedoutputstream.c (Diff revision 1)
     
     
    Show all issues

    Space before =.

  5. 
      
grim
QuLogic
  1. 
      
  2. libpurple/queuedoutputstream.h (Diff revision 2)
     
     
    Show all issues

    This doesn't match the argument name. You should have got a warning about that.

  3. 
      
grim
QuLogic
  1. Ship It!
  2. 
      
grim
Review request changed
Status:
Completed