Clean up Finch preferences page

Review Request #2416 — Created April 3, 2023 and submitted April 3, 2023, 5:03 p.m.

Information
Owner:	QuLogic
Repository:	pidgin/pidgin
Branch:	default
Bugs:
Depends On:
Reviewers
Groups:	pidgin
People:

Description

Instead of mixing two strings into a GList, use a PurpleKeyValuePair.

Use struct initializers and G_N_ELEMENTS.

Rename lv to choices for clarity.

Change from PurpleRequestFieldList to PurpleRequestFieldChoice as we don't need multiple selection.

Testing Done

Opened Finch, opened Preferences, saved them, and exited Finch. Re-opened Prefs and confirmed they were saved and as expected.

Commits

Summary
Cleanup Finch preferences page

Screenshots

Issues

Description	From	Last Updated

Change Summary:

Change from PurpleRequestFieldList to PurpleRequestFieldChoice.

Description:

		Instead of mixing two strings into a `GList`, use a `PurpleKeyValuePair`.
		Use struct initializers and `G_N_ELEMENTS`.
~		Rename `lv` to `choices` for clarity.
	~	Rename `lv` to `choices` for clarity.
	+	Change from `PurpleRequestFieldList` to `PurpleRequestFieldChoice` as we don't need multiple selection.

Testing Done:

~		Opened Finch, opened Preferences, saved them, and exited Finch.
	~	Opened Finch, opened Preferences, saved them, and exited Finch. Re-opened Prefs and confirmed they were saved and as expected.

Commits:

	Summary
-	Cleanup Finch preferences page
+	Cleanup Finch preferences page

Diff:

Revision 2 (+290 -186)

Show changes

	finch/gntprefs.c
	finch/gntrequest.c

Ship it!

```
Ship It!
```
```
Nice work, thanks!!
```

You have a pending review.

Review Board 5.0.4