Support for blobless repositories

jairbubbles · jairbubbles · commit c423ad191d93 · 2021-09-12T10:05:03.000+02:00
diff --git a/LibGit2Sharp/Blob.cs b/LibGit2Sharp/Blob.cs
@@ -12,6 +12,7 @@ public class Blob : GitObject
     {
         private readonly ILazy<Int64> lazySize;
         private readonly ILazy<bool> lazyIsBinary;
+        private readonly ILazy<bool> lazyIsDownloaded;
 
         /// <summary>
         /// Needed for mocking purposes.
@@ -24,6 +25,7 @@ internal Blob(Repository repo, ObjectId id)
         {
             lazySize = GitObjectLazyGroup.Singleton(repo, id, Proxy.git_blob_rawsize);
             lazyIsBinary = GitObjectLazyGroup.Singleton(repo, id, Proxy.git_blob_is_binary);
+            lazyIsDownloaded = GitObjectLazyGroup.Singleton(repo, id, handle => handle != null);
         }
 
         /// <summary>
@@ -33,12 +35,18 @@ internal Blob(Repository repo, ObjectId id)
         /// can be used.
         /// </para>
         /// </summary>
-        public virtual long Size { get { return lazySize.Value; } }
+        public virtual long Size => lazySize.Value;
 
         /// <summary>
         ///  Determine if the blob content is most certainly binary or not.
         /// </summary>
-        public virtual bool IsBinary { get { return lazyIsBinary.Value; } }
+        public virtual bool IsBinary => lazyIsBinary.Value;
+
+
+        /// <summary>
+        ///  Determine if the blob content was downloaded (happens with partially cloned repositories)
+        /// </summary>
+        public virtual bool IsDownloaded => lazyIsDownloaded.Value;
 
         /// <summary>
         /// Gets the blob content in a <see cref="Stream"/>.
diff --git a/LibGit2Sharp/Core/Proxy.cs b/LibGit2Sharp/Core/Proxy.cs
@@ -71,9 +71,11 @@ public static unsafe ObjectId git_blob_create_from_workdir(RepositoryHandle repo
 
         public static unsafe UnmanagedMemoryStream git_blob_filtered_content_stream(RepositoryHandle repo, ObjectId id, string path, bool check_for_binary_data)
         {
-            var buf = new GitBuf();
             var handle = new ObjectSafeWrapper(id, repo).ObjectPtr;
+            if (handle == null)
+                return null;
 
+            var buf = new GitBuf();
             return new RawContentStream(handle, h =>
             {
                 Ensure.ZeroResult(NativeMethods.git_blob_filtered_content(buf, h, path, check_for_binary_data));
@@ -86,16 +88,25 @@ public static unsafe UnmanagedMemoryStream git_blob_filtered_content_stream(Repo
         public static unsafe UnmanagedMemoryStream git_blob_rawcontent_stream(RepositoryHandle repo, ObjectId id, Int64 size)
         {
             var handle = new ObjectSafeWrapper(id, repo).ObjectPtr;
+            if (handle == null)
+                return null;
+
             return new RawContentStream(handle, h => NativeMethods.git_blob_rawcontent(h), h => size);
         }
 
         public static unsafe long git_blob_rawsize(ObjectHandle obj)
         {
+            if (obj == null)
+                return 0;
+
             return NativeMethods.git_blob_rawsize(obj);
         }
 
         public static unsafe bool git_blob_is_binary(ObjectHandle obj)
         {
+            if (obj == null)
+                return false;
+
             int res = NativeMethods.git_blob_is_binary(obj);
             Ensure.BooleanResult(res);
 

Original file line number	Diff line number	Diff line change
`@@ -71,9 +71,11 @@ public static unsafe ObjectId git_blob_create_from_workdir(RepositoryHandle repo`
`71`	`71`
`72`	`72`	`public static unsafe UnmanagedMemoryStream git_blob_filtered_content_stream(RepositoryHandle repo, ObjectId id, string path, bool check_for_binary_data)`
`73`	`73`	`{`
`74`		`- var buf = new GitBuf();`
`75`	`74`	`var handle = new ObjectSafeWrapper(id, repo).ObjectPtr;`
	`75`	`+ if (handle == null)`
	`76`	`+ return null;`
`76`	`77`
	`78`	`+ var buf = new GitBuf();`
`77`	`79`	`return new RawContentStream(handle, h =>`
`78`	`80`	`{`
`79`	`81`	`Ensure.ZeroResult(NativeMethods.git_blob_filtered_content(buf, h, path, check_for_binary_data));`
`@@ -86,16 +88,25 @@ public static unsafe UnmanagedMemoryStream git_blob_filtered_content_stream(Repo`
`86`	`88`	`public static unsafe UnmanagedMemoryStream git_blob_rawcontent_stream(RepositoryHandle repo, ObjectId id, Int64 size)`
`87`	`89`	`{`
`88`	`90`	`var handle = new ObjectSafeWrapper(id, repo).ObjectPtr;`
	`91`	`+ if (handle == null)`
	`92`	`+ return null;`
	`93`	`+`
`89`	`94`	`return new RawContentStream(handle, h => NativeMethods.git_blob_rawcontent(h), h => size);`
`90`	`95`	`}`
`91`	`96`
`92`	`97`	`public static unsafe long git_blob_rawsize(ObjectHandle obj)`
`93`	`98`	`{`
	`99`	`+ if (obj == null)`
	`100`	`+ return 0;`
	`101`	`+`
`94`	`102`	`return NativeMethods.git_blob_rawsize(obj);`
`95`	`103`	`}`
`96`	`104`
`97`	`105`	`public static unsafe bool git_blob_is_binary(ObjectHandle obj)`
`98`	`106`	`{`
	`107`	`+ if (obj == null)`
	`108`	`+ return false;`
	`109`	`+`
`99`	`110`	`int res = NativeMethods.git_blob_is_binary(obj);`
`100`	`111`	`Ensure.BooleanResult(res);`
`101`	`112`