Summary

Expose the GPluginManager class

Review Request #689 — Created May 31, 2021 and submitted June 18, 2021, 5: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, 5:03 a.m.
Maybe should add all the braces on these?	QuLogic	June 18, 2021, 5: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)
Show all issues
```
a path
```
gplugin/gplugin-manager.c (Diff revision 2)
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!
```

Review Board 6.0.2

Expose the GPluginManager class

Information

Reviewers

Change Summary:

Commit:

Diff:

Commit:

Diff:

Status: Closed (submitted)

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

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