Summary

Clean up the native loader

Review Request #2471 — Created June 16, 2023 and submitted June 22, 2023, 3 a.m.

Information

Owner

grim

Repository

gplugin/gplugin

Branch

default

Bugs

Depends On

Reviewers

Groups

pidgin

People

Description

The native loader code was written a very long time ago before I knew about
g_propagate_error and did some weird things with trying to use memory from the
module that it just unloaded. This takes care of all of that.

Testing Done

Ran the unit tests without issue

Commits

Summary	ID
Clean up the native loader The native loader code was written a very long time ago before I knew about g_propagate_error and did some weird things with trying to use memory from the module that it just unloaded. This takes care of all of that.	d161e935f5027613a2f47102ad8f9b7d8fd3c495

Issues

Description	From	Last Updated
s/with invalid/will invalidate?	ivanhoe	June 20, 2023, 6:46 p.m.

gplugin/gplugin-native-loader.c (Diff revision 1)
The issue has been resolved. Show all issues
```
s/with invalid/will invalidate?
```

Change Summary:

Fix typos, not sure why the italicize lose part of the d.. but if you copy paste into another text field it is correct.

Commits:

	Summary	ID
	Clean up the native loader The native loader code was written a very long time ago before I knew about g_propagate_error and did some weird things with trying to use memory from the module that it just unloaded. This takes care of all of that.	906a89f95016fc74215bea59ee4cc74f10cd8dd2
	Clean up the native loader The native loader code was written a very long time ago before I knew about g_propagate_error and did some weird things with trying to use memory from the module that it just unloaded. This takes care of all of that.	d161e935f5027613a2f47102ad8f9b7d8fd3c495

Diff:

Revision 2 (+170 -150)

Show changes

	gplugin/gplugin-native-loader.c
	gplugin/tests/test-native-loader.c

Ship it!

```
Ship It!
```

Status:: Completed