Port gnttinyurl plugin to libsoup3

Review Request #1961 — Created Oct. 25, 2022 and submitted

Information
QuLogic
pidgin/pidgin
default
1960
Reviewers
pidgin 
This covers both adding the SoupMessage to an existing callback data structure, and using SoupMessage as the callback data with "real" callback data attached to it.
Compiled against libsoup2 and libsoup3.
Summary 
Port gnttinyurl plugin to libsoup3
Description From Last Updated

If we don't finish, the result just sits in limbo no?

 grimgrim

"cancel it" perhaps?

 grimgrim

where does g_cancellable_cancel come from?

 grimgrim
grim
grim 
  1. 
  
    
 
    

 
    
  


    
 
    
 
      

  

 
    


    

 
    

    2. 

















  2. 
 
    
  
  
    
  
 
    
  
    
   
    
    
     
      finch/plugins/gnttinyurl/gnttinyurl.c
      


       (Diff revision 1)


      
     
    
    
       
   
    
     
    

    
     
    

       
  
    
 
    

   
    
    
    



    
 
    
  
    
   
   
    
    
     
     
     
     
     
    
    Show all issues
   
  
    
 
    

    



    
    If we don't finish, the result just sits in limbo no?
    
  
    
 
    

 
    
  


    
 
    
 
      

  


    1. 
 
 
      
 
      
  
 
      
 
      
  
      libsoup uses GTask for async results, and g_task_propagate_pointer "steals" the result from the task. That means if you don't call finish, the task still owns it and will free the result with the GDestroyNotify that was given to g_task_return_pointer. So I don't think we need to call finish always.
      
 
      

      2. 



    2. 
 
 
      
 
      
  
 
      
 
      
  
      Right right, this coming back to me now. The issue I was remembering was when we were trying to early fail in the _async method.
      
 
      

      3. 


 
    


    

 
    

    3. 






  3. 
 
    
  
  
    
  
 
    
  
    
   
    
    
     
      finch/plugins/gnttinyurl/gnttinyurl.c
      


       (Diff revision 1)


      
     
    
    
       
   
    
     
    

    
     
    

       
  
    
 
    

   
    
    
    



    
 
    
  
    
   
   
    
    
     
     
     
     
     
    
    Show all issues
   
  
    
 
    

    



    
    "cancel it" perhaps?
    
  
    
 
    

 
    
  


    
 
    
 
      

  

 
    


    

 
    

    4. 






  4. 
 
    
  
  
    
  
 
    
  
    
   
    
    
     
      finch/plugins/gnttinyurl/gnttinyurl.c
      


       (Diff revision 1)


      
     
    
    
       
   
    
     
    

    
     
    

       
  
    
 
    

   
    
    
    



    
 
    
  
    
   
   
    
    
     
     
     
     
     
    
    Show all issues
   
  
    
 
    

    



    
    where does g_cancellable_cancel come from?
    
  
    
 
    

 
    
  



 
    

    5. 




  5. 
 
    
  
    
   
    
  
    
 
    

 
    
  


    
 
    
 
      

  

 
    


    

 
    

    6. 




  

 
















 



 

  

   




QuLogic




   

   

  

 



 















 





 

  

   




grim




   

   







  

 



 

  


   

   

    
 
grim


    
   


  

  

  



    




  1. 
 
    
  
    
   
    Ship It!
    
  
    
 
    

 
    
  


    
 
    
 
      

  

 
    


    

 
    

    2. 







  2. 
 
    
  
  
    
  
   
    
    
    




    
    Great work, thanks!!
    
  
    
 
    

 
    
  


    
 
    
 
      

  

 
    


    

 
    

    3. 













  3. 
 
    
  
    
   
    
  
    
 
    

 
    
  


    
 
    
 
      

  

 
    


    

 
    

    4. 




  

 
















 



 

  

   




grim




   

   

  

 



 

  


   

   

    
 Review request changed

    
   


  

  

  






 

  Status:
  

  Closed (submitted)

  
 







 


 










  

 









     

    

   

  

  

   
   Loading...