repo: Drop ostree_repo_load_variant_c API

Originally we had this to avoid forcing callers to malloc() if they
had a csum, but nowadays we have in-place conversion APIs that are
fast enough.

doc/ostree-sections.txt

@@ -90,7 +90,6 @@ ostree_repo_resolve_rev
 ostree_repo_write_ref
 ostree_repo_write_refspec
 ostree_repo_list_refs
-ostree_repo_load_variant_c
 ostree_repo_load_variant
 ostree_repo_load_variant_if_exists
 ostree_repo_load_file

src/libostree/ostree-repo-file.c

@@ -169,6 +169,7 @@ do_resolve_commit (OstreeRepoFile  *self,
   gs_unref_variant GVariant *root_metadata = NULL;
   gs_unref_variant GVariant *tree_contents_csum_v = NULL;
   gs_unref_variant GVariant *tree_metadata_csum_v = NULL;
+  char tmp_checksum[65];
 
   g_assert (self->parent == NULL);
 
@@ -178,16 +179,20 @@ do_resolve_commit (OstreeRepoFile  *self,
 
   /* PARSE OSTREE_OBJECT_TYPE_COMMIT */
   g_variant_get_child (commit, 6, "@ay", &tree_contents_csum_v);
-  g_variant_get_child (commit, 7, "@ay", &tree_metadata_csum_v);
 
-  if (!ostree_repo_load_variant_c (self->repo, OSTREE_OBJECT_TYPE_DIR_TREE,
+  ostree_checksum_inplace_from_bytes (g_variant_get_data (tree_contents_csum_v), tmp_checksum);
-                                   ostree_checksum_bytes_peek (tree_contents_csum_v),
+
-                                   &root_contents, error))
+  if (!ostree_repo_load_variant (self->repo, OSTREE_OBJECT_TYPE_DIR_TREE,
+                                 tmp_checksum,
+                                 &root_contents, error))
     goto out;
 
-  if (!ostree_repo_load_variant_c (self->repo, OSTREE_OBJECT_TYPE_DIR_META,
+  g_variant_get_child (commit, 7, "@ay", &tree_metadata_csum_v);
-                                   ostree_checksum_bytes_peek (tree_metadata_csum_v),
+  ostree_checksum_inplace_from_bytes (g_variant_get_data (tree_metadata_csum_v), tmp_checksum);
-                                   &root_metadata, error))
+
+  if (!ostree_repo_load_variant (self->repo, OSTREE_OBJECT_TYPE_DIR_META,
+                                 tmp_checksum,
+                                 &root_metadata, error))
     goto out;
   
   self->tree_metadata = root_metadata;

src/libostree/ostree-repo.c

@@ -2584,37 +2584,6 @@ ostree_repo_query_object_storage_size (OstreeRepo           *self,
   return ret;
 }
 
-/**
- * ostree_repo_load_variant_c:
- * @self: Repo
- * @objtype: Expected object type
- * @csum: Binary checksum
- * @out_variant: (out): (transfer full): Metadata object
- * @error: Error
- * 
- * Load the metadata object @csum of type @objtype, storing the
- * result in @out_variant.
- */
-gboolean
-ostree_repo_load_variant_c (OstreeRepo          *self,
-                            OstreeObjectType     objtype,
-                            const guchar        *csum, 
-                            GVariant           **out_variant,
-                            GError             **error)
-{
-  gboolean ret = FALSE;
-  gs_free char *checksum = NULL;
-
-  checksum = ostree_checksum_from_bytes (csum);
-
-  if (!ostree_repo_load_variant (self, objtype, checksum, out_variant, error))
-    goto out;
-
-  ret = TRUE;
- out:
-  return ret;
-}
-
 /**
  * ostree_repo_load_variant_if_exists:
  * @self: Repo

src/libostree/ostree-repo.h

@@ -181,12 +181,6 @@ gboolean      ostree_repo_list_refs (OstreeRepo       *self,
                                      GCancellable     *cancellable,
                                      GError          **error);
 
-gboolean      ostree_repo_load_variant_c (OstreeRepo  *self,
-                                          OstreeObjectType objtype,
-                                          const guchar  *csum,       
-                                          GVariant     **out_variant,
-                                          GError       **error);
-
 gboolean      ostree_repo_load_variant (OstreeRepo  *self,
                                         OstreeObjectType objtype,
                                         const char    *sha256,