From mboxrd@z Thu Jan  1 00:00:00 1970
From: Michael Tremer <michael.tremer@ipfire.org>
To: location@lists.ipfire.org
Subject: Re: [PATCH 1/3] perl: Add database_countries() function.
Date: Fri, 21 Aug 2020 10:58:58 +0100
Message-ID: <ABE31E64-72C7-45E8-BB62-ABAABF5F13ED@ipfire.org>
In-Reply-To: <20200820172856.6983-1-stefan.schantl@ipfire.org>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============0588847152240522592=="
List-Id: <location.lists.ipfire.org>

--===============0588847152240522592==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable

Thank you. I merged these. The tests run through.

I won=E2=80=99t tag a release just yet and will wait for the remaining work t=
o be completed.

Best,
-Michael

> On 20 Aug 2020, at 18:28, Stefan Schantl <stefan.schantl(a)ipfire.org> wrot=
e:
>=20
> This function is used to the stored countries of a database, which
> easily can be assigned to an array.
>=20
> Signed-off-by: Stefan Schantl <stefan.schantl(a)ipfire.org>
> ---
> src/perl/Location.xs | 42 ++++++++++++++++++++++++++++++++++++++++--
> 1 file changed, 40 insertions(+), 2 deletions(-)
>=20
> diff --git a/src/perl/Location.xs b/src/perl/Location.xs
> index 5693744..3c347db 100644
> --- a/src/perl/Location.xs
> +++ b/src/perl/Location.xs
> @@ -6,11 +6,10 @@
> #include <stdio.h>
> #include <string.h>
>=20
> -
> #include <loc/libloc.h>
> #include <loc/database.h>
> #include <loc/network.h>
> -
> +#include <loc/country.h>
>=20
> MODULE =3D Location		PACKAGE =3D Location
>=20
> @@ -119,6 +118,45 @@ get_license(db)
> 	OUTPUT:
> 		RETVAL
>=20
> +void
> +database_countries(db)
> +	struct loc_database* db;
> +
> +	PPCODE:
> +		// Create Database enumerator
> +		struct loc_database_enumerator* enumerator;
> +		int err =3D loc_database_enumerator_new(&enumerator, db, LOC_DB_ENUMERAT=
E_COUNTRIES);
> +
> +		if (err) {
> +			croak("Could not create a database enumerator\n");
> +		}
> +
> +		// Init and enumerate first country.
> +		struct loc_country* country;
> +		err =3D loc_database_enumerator_next_country(enumerator, &country);
> +		if (err) {
> +			croak("Could not enumerate next country\n");
> +		}
> +
> +		while (country) {
> +			// Extract the country code.
> +			const char* ccode =3D loc_country_get_code(country);
> +
> +			// Push country code.
> +			XPUSHs(sv_2mortal(newSVpv(ccode, 2)));
> +
> +			// Unref country pointer.
> +			loc_country_unref(country);
> +
> +			// Enumerate next item.
> +			err =3D loc_database_enumerator_next_country(enumerator, &country);
> +			if (err) {
> +				croak("Could not enumerate next country\n");
> +			}
> +		}
> +
> +		loc_database_enumerator_unref(enumerator);
> +
> #
> # Lookup functions
> #
> --=20
> 2.20.1
>=20


--===============0588847152240522592==--