[virt-tools-list] [libosinfo] Micro coding-style fixes
Richard W.M. Jones
rjones at redhat.com
Thu Jun 23 12:56:15 UTC 2011
On Thu, Jun 23, 2011 at 12:20:39AM +0300, Zeeshan Ali wrote:
> From: "Zeeshan Ali (Khattak)" <zeeshanak at gnome.org>
>
> - Remove redundant newlines.
> - Add newlines where needed.
> ---
> osinfo/osinfo_loader.c | 22 ++--------------------
> 1 files changed, 2 insertions(+), 20 deletions(-)
>
> diff --git a/osinfo/osinfo_loader.c b/osinfo/osinfo_loader.c
> index 17338b8..ea93a63 100644
> --- a/osinfo/osinfo_loader.c
> +++ b/osinfo/osinfo_loader.c
> @@ -64,7 +64,6 @@ osinfo_loader_finalize (GObject *object)
> G_OBJECT_CLASS (osinfo_loader_parent_class)->finalize (object);
> }
>
> -
> /* Init functions */
> static void
> osinfo_loader_class_init (OsinfoLoaderClass *klass)
> @@ -76,7 +75,6 @@ osinfo_loader_class_init (OsinfoLoaderClass *klass)
> g_type_class_add_private (klass, sizeof (OsinfoLoaderPrivate));
> }
>
> -
> static void
> osinfo_loader_init (OsinfoLoader *loader)
> {
> @@ -100,7 +98,6 @@ OsinfoLoader *osinfo_loader_new(void)
> return g_object_new(OSINFO_TYPE_LOADER, NULL);
> }
>
> -
> #define OSINFO_ERROR(err, msg) \
> g_set_error_literal((err), g_quark_from_static_string("libosinfo"), 0, (msg));
>
> @@ -144,7 +141,6 @@ osinfo_loader_nodeset(const char *xpath,
> return (ret);
> }
>
> -
> static gchar *
> osinfo_loader_string(const char *xpath,
> xmlXPathContextPtr ctxt,
> @@ -171,7 +167,6 @@ osinfo_loader_string(const char *xpath,
> return ret;
> }
>
> -
> static void osinfo_loader_entity(OsinfoLoader *loader,
> OsinfoEntity *entity,
> const gchar *const *keys,
> @@ -219,7 +214,6 @@ static void osinfo_loader_entity(OsinfoLoader *loader,
> g_free(custom);
> }
>
> -
> static OsinfoDevice *osinfo_loader_get_device(OsinfoLoader *loader,
> const gchar *id)
> {
> @@ -232,7 +226,6 @@ static OsinfoDevice *osinfo_loader_get_device(OsinfoLoader *loader,
> return dev;
> }
>
> -
> static OsinfoOs *osinfo_loader_get_os(OsinfoLoader *loader,
> const gchar *id)
> {
> @@ -245,7 +238,6 @@ static OsinfoOs *osinfo_loader_get_os(OsinfoLoader *loader,
> return os;
> }
>
> -
> static OsinfoPlatform *osinfo_loader_get_platform(OsinfoLoader *loader,
> const gchar *id)
> {
> @@ -258,7 +250,6 @@ static OsinfoPlatform *osinfo_loader_get_platform(OsinfoLoader *loader,
> return platform;
> }
>
> -
> static void osinfo_loader_device(OsinfoLoader *loader,
> xmlXPathContextPtr ctxt,
> xmlNodePtr root,
> @@ -284,7 +275,6 @@ static void osinfo_loader_device(OsinfoLoader *loader,
> osinfo_loader_entity(loader, OSINFO_ENTITY(device), keys, ctxt, root, err);
> }
>
> -
> static void osinfo_loader_device_link(OsinfoLoader *loader,
> OsinfoEntity *entity,
> const gchar *xpath,
> @@ -332,7 +322,6 @@ static void osinfo_loader_device_link(OsinfoLoader *loader,
> g_free(related);
> }
>
> -
> static void osinfo_loader_product_relshp(OsinfoLoader *loader,
> OsinfoProduct *product,
> OsinfoProductRelationship relshp,
> @@ -367,7 +356,6 @@ static void osinfo_loader_product_relshp(OsinfoLoader *loader,
> g_free(related);
> }
>
> -
> static void osinfo_loader_product(OsinfoLoader *loader,
> OsinfoProduct *product,
> xmlXPathContextPtr ctxt,
> @@ -444,7 +432,6 @@ static void osinfo_loader_platform(OsinfoLoader *loader,
> return;
> }
>
> -
> static void osinfo_loader_deployment(OsinfoLoader *loader,
> xmlXPathContextPtr ctxt,
> xmlNodePtr root,
> @@ -492,7 +479,6 @@ static void osinfo_loader_deployment(OsinfoLoader *loader,
> osinfo_db_add_deployment(loader->priv->db, deployment);
> }
>
> -
> static void osinfo_loader_os(OsinfoLoader *loader,
> xmlXPathContextPtr ctxt,
> xmlNodePtr root,
> @@ -524,7 +510,6 @@ static void osinfo_loader_os(OsinfoLoader *loader,
> return;
> }
>
> -
> static void osinfo_loader_root(OsinfoLoader *loader,
> xmlXPathContextPtr ctxt,
> xmlNodePtr root,
> @@ -616,7 +601,6 @@ static void osinfo_loader_root(OsinfoLoader *loader,
> g_free(devices);
> }
>
> -
> static void
> catchXMLError(void *ctx, const char *msg ATTRIBUTE_UNUSED, ...)
> {
> @@ -807,6 +791,7 @@ osinfo_loader_process_file_reg_usb(OsinfoLoader *loader,
> "usb",
> err);
> }
> +
> static void
> osinfo_loader_process_file_reg_pci(OsinfoLoader *loader,
> GFile *file,
> @@ -879,7 +864,6 @@ osinfo_loader_process_file_dir(OsinfoLoader *loader,
> g_object_unref(ents);
> }
>
> -
> static void
> osinfo_loader_process_file(OsinfoLoader *loader,
> GFile *file,
> @@ -921,7 +905,6 @@ osinfo_loader_process_file(OsinfoLoader *loader,
> g_object_unref(info);
> }
>
> -
> /**
> * osinfo_loader_get_db:
> * @loader: the loader object
> @@ -937,7 +920,6 @@ OsinfoDb *osinfo_loader_get_db(OsinfoLoader *loader)
> return loader->priv->db;
> }
>
> -
> /**
> * osinfo_loader_process_path:
> * @loader: the loader object
> @@ -960,7 +942,6 @@ void osinfo_loader_process_path(OsinfoLoader *loader,
> g_object_unref(file);
> }
>
> -
> /**
> * osinfo_loader_process_uri:
> * @loader: the loader object
> @@ -982,6 +963,7 @@ void osinfo_loader_process_uri(OsinfoLoader *loader,
> err);
> g_object_unref(file);
> }
> +
> /*
> * Local variables:
> * indent-tabs-mode: nil
All good, ACK.
Rich.
--
Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones
Read my programming blog: http://rwmj.wordpress.com
Fedora now supports 80 OCaml packages (the OPEN alternative to F#)
http://cocan.org/getting_started_with_ocaml_on_red_hat_and_fedora
More information about the virt-tools-list
mailing list