simple_git 0.1.2 → 0.1.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: bdb1902dec8bb172009fcc2248ef4bd603fdd179ef9068c0d6811e813363a132
-  data.tar.gz: 5bb013a78bc27ed4a7969eb438597016f3e525347383f814caada7ba80c9b206
+  metadata.gz: dfe7663686ad5df1231d65e3dd3f25f0460751d851ab5f58979f467e391d99bb
+  data.tar.gz: 2cb02766d0ce2ce498059cd6e68419703fb383bdec6e5cad706e98e64ab78171
 SHA512:
-  metadata.gz: 66407df6d074172e70dae66d6cb5a20693ff1386997e5e28440296ce7743530a0ffa82360c5c950d58446a459ce5a7dbb7ecc90f2468e3f5b60d3e57a971c055
-  data.tar.gz: 3383ffa03fd3273bba8eb6e0a37dee14224b7af77c806f61c8dea121fdae19e4143b4e3d11f84ae4e1919f7bb42710acbbfde9e03a5a38fe00c858da84009101
+  metadata.gz: 1f947b79828cd0d70969d95002cae43b0c8d3785ede94acb4141ac45ca393622fe6781e058936eafd848cf5cfc6b3924bf5f04ed8455abcc30b18ad050308f9d
+  data.tar.gz: 76126eefebe9ba70cb25a8d4429884c54ca1d7ad9ac2e97edcfd72ea51047d6167f8bf08441a81c9803b051a77730b25a8476373cfdb0c490009e9f5843cd961

data/README.md

@@ -15,7 +15,7 @@ puts "Current HEAD: #{repo.revparse('HEAD')}\n"
 walk.sort(:GIT_SORT_TOPOLOGICAL)
 walk.push_head
 
-puts 'Last 25 commits:\n'
+puts "Last 25 commits:\n"
 
 walk.take(25).each do |c|
   next if c.parent_count != 1

data/lib/simple_git/commit.rb

@@ -4,7 +4,11 @@ module SimpleGit
 
     def initialize(repo, oid)
       wrapper = CommitWrapper.new
-      Git2.git_commit_lookup(wrapper, repo.ptr, oid.ptr)
+      ret = Git2.git_commit_lookup(wrapper, repo.ptr, oid.ptr)
+      if ret != 0
+        error = Git2::GitError.new(Git2.giterr_last)
+        raise ArgumentError, error[:message].read_string
+      end
 
       @repo = repo
       @oid = oid.to_s
@@ -15,7 +19,11 @@ module SimpleGit
 
     def parent(n)
       wrapper = CommitWrapper.new
-      Git2.git_commit_parent(wrapper, @ptr, n)
+      ret = Git2.git_commit_parent(wrapper, @ptr, n)
+      if ret != 0
+        error = Git2::GitError.new(Git2.giterr_last)
+        raise ArgumentError, error[:message].read_string
+      end
 
       c = Commit.allocate
       c.ptr = wrapper[:commit]
@@ -37,7 +45,7 @@ module SimpleGit
     end
 
     def message
-      @message ||= Git2.git_commit_message(@ptr).read_string
+      @message ||= Git2.git_commit_message(@ptr)&.read_string
     end
 
     def oid

data/lib/simple_git/diff.rb

@@ -4,7 +4,11 @@ module SimpleGit
 
     def from_trees(repo, old_tree, new_tree, options)
       wrapper = DiffWrapper.new
-      Git2.git_diff_tree_to_tree(wrapper, repo.ptr, old_tree.ptr, new_tree.ptr, options.ptr)
+      ret = Git2.git_diff_tree_to_tree(wrapper, repo.ptr, old_tree.ptr, new_tree.ptr, options.ptr)
+      if ret != 0
+        error = Git2::GitError.new(Git2.giterr_last)
+        raise ArgumentError, error[:message].read_string
+      end
 
       @ptr = wrapper[:diff]
       ObjectSpace.define_finalizer(self, self.class.finalize(@ptr))

data/lib/simple_git/diff_stat.rb

@@ -4,7 +4,11 @@ module SimpleGit
 
     def initialize(diff)
       wrapper = DiffStatWrapper.new
-      Git2.git_diff_get_stats(wrapper, diff.ptr)
+      ret = Git2.git_diff_get_stats(wrapper, diff.ptr)
+      if ret != 0
+        error = Git2::GitError.new(Git2.giterr_last)
+        raise ArgumentError, error[:message].read_string
+      end
 
       @ptr = wrapper[:stat]
       ObjectSpace.define_finalizer(self, self.class.finalize(@ptr))

data/lib/simple_git/repository.rb

@@ -17,7 +17,11 @@ module SimpleGit
 
     def revparse(refspec)
       wrapper = SimpleGit::Object::ObjectWrapper.new
-      Git2.git_revparse_single(wrapper, @ptr, refspec)
+      ret = Git2.git_revparse_single(wrapper, @ptr, refspec)
+      if ret != 0
+        error = Git2::GitError.new(Git2.giterr_last)
+        raise ArgumentError, error[:message].read_string
+      end
 
       Object.new.from_wrapper(wrapper)
     end

data/lib/simple_git/revwalk.rb

@@ -6,7 +6,11 @@ module SimpleGit
 
     def initialize(repo)
       wrapper = RevwalkWrapper.new
-      Git2.git_revwalk_new(wrapper, repo.ptr)
+      ret = Git2.git_revwalk_new(wrapper, repo.ptr)
+      if ret != 0
+        error = Git2::GitError.new(Git2.giterr_last)
+        raise ArgumentError, error[:message].read_string
+      end
 
       @repo = repo
       @ptr = wrapper[:revwalk]

data/lib/simple_git/tree.rb

@@ -4,7 +4,11 @@ module SimpleGit
     
     def from_commit(commit)
       wrapper = TreeWrapper.new
-      Git2.git_commit_tree(wrapper, commit.ptr)
+      ret = Git2.git_commit_tree(wrapper, commit.ptr)
+      if ret != 0
+        error = Git2::GitError.new(Git2.giterr_last)
+        raise ArgumentError, error[:message].read_string
+      end
 
       @ptr = wrapper[:tree]
       ObjectSpace.define_finalizer(self, self.class.finalize(@ptr))

data/lib/simple_git/version.rb

@@ -1,3 +1,3 @@
 module SimpleGit
-  VERSION = '0.1.2'
+  VERSION = '0.1.3'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: simple_git
 version: !ruby/object:Gem::Version
-  version: 0.1.2
+  version: 0.1.3
 platform: ruby
 authors:
 - Liam P. White