Change loader registration to work on an instance rather than a GType

Review Request #899 — Created Aug. 16, 2021 and submitted

Information

gplugin/gplugin
default

Reviewers

Change loader registration to work on an instance rather than a GType

Ran the unit tests in valgrind and verifed there was no new memory leaks in the loader registration. Right now the plugins themselfs leak a reference, but I'll be cleaning that up shortly.

Summary ID
Change loader registration to work on an instance rather than a GType
3038b542c0e0adc0168a96dcf558881a20895088
Description From Last Updated

Should this not use gplugin_native_loader_new?

QuLogicQuLogic

"So we'll" is doubled

QuLogicQuLogic

Shouldn't this be !g_str_equal?

QuLogicQuLogic

instance

QuLogicQuLogic
QuLogic
  1. 
      
  2. This seems fine, but I wonder what precipitated the change?

    1. just trying to standardize on instances like we have in pidgin rather than types.

  3. gplugin/gplugin-manager.c (Diff revision 1)
     
     
     
     
     
     
     
    Show all issues

    Should this not use gplugin_native_loader_new?

  4. gplugin/gplugin-manager.c (Diff revision 1)
     
     
    Show all issues

    Shouldn't this be !g_str_equal?

  5. 
      
grim
QuLogic
  1. 
      
  2. gplugin/gplugin-manager.c (Diff revisions 1 - 2)
     
     
     
    Show all issues

    "So we'll" is doubled

  3. gplugin/gplugin-native-loader.c (Diff revisions 1 - 2)
     
     
    Show all issues

    instance

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