Summary

Expose the GPluginManager class

Review Request #689 — Created May 31, 2021 and submitted June 18, 2021, 12:16 a.m.

Information

Owner

grim

Repository

gplugin/gplugin

Branch

default

Bugs

Depends On

Commit

08b937449340

Reviewers

Groups

pidgin

People

Description

This is the first step in converting to the manager paradigm we've been using
in pidgin.

Testing Done

Ran the unit tests. This currently breaks the vapi bindings for the manager, but nothing is using them right now so it's okay and will be fixed in a follow up pull request.

Issues

Description	From	Last Updated
a path	QuLogic	June 18, 2021, 12:03 a.m.
Maybe should add all the braces on these?	QuLogic	June 18, 2021, 12:04 a.m.

Change Summary:

ran ninja clang-format

Commit:

f8312424a560

b3dadac2c1eb

Diff:

Revision 2 (+284 -230)

Show changes

	gplugin/gplugin-manager.h
	gplugin/gplugin-manager.c
	gplugin-gtk/gplugin-gtk-store.c
	gplugin/tests/test-signals.c

gplugin/gplugin-manager.c (Diff revision 2)
The issue has been resolved. Show all issues
```
a path
```
gplugin/gplugin-manager.c (Diff revision 2)
The issue has been resolved. Show all issues
```
Maybe should add all the braces on these?
```

Commit:

b3dadac2c1eb

08b937449340

Diff:

Revision 3 (+316 -246)

Show changes

	gplugin/gplugin-manager.h
	gplugin/gplugin-manager.c
	gplugin-gtk/gplugin-gtk-store.c
	gplugin/tests/test-signals.c

Ship it!

```
Ship It!
```

Status:: Completed

gplugin/gplugin-manager.c (Diff revision 2)

gplugin/gplugin-manager.c (Diff revision 2)