[gitg] No need to store repository with StageStatusSubmodule



commit b6d757f05a26da9cdfa6d1360f6abc035ad84931
Author: Jesse van den Kieboom <jessevdk gnome org>
Date:   Thu Jul 30 20:37:12 2015 +0200

    No need to store repository with StageStatusSubmodule

 libgitg/gitg-stage-status-enumerator.vala |   10 ++++------
 1 files changed, 4 insertions(+), 6 deletions(-)
---
diff --git a/libgitg/gitg-stage-status-enumerator.vala b/libgitg/gitg-stage-status-enumerator.vala
index c3e1a52..709698b 100644
--- a/libgitg/gitg-stage-status-enumerator.vala
+++ b/libgitg/gitg-stage-status-enumerator.vala
@@ -114,7 +114,6 @@ public class StageStatusFile : Object, StageStatusItem
 
 public class StageStatusSubmodule : Object, StageStatusItem
 {
-       private Ggit.Repository d_repository;
        private Ggit.Submodule d_submodule;
        private string d_path;
        private Ggit.SubmoduleStatus d_flags;
@@ -140,14 +139,13 @@ public class StageStatusSubmodule : Object, StageStatusItem
                  Ggit.SubmoduleStatus.WD_INDEX_MODIFIED
                | Ggit.SubmoduleStatus.WD_WD_MODIFIED;
 
-       public StageStatusSubmodule(Repository repository,
-                                   Ggit.Submodule submodule)
+       public StageStatusSubmodule(Ggit.Submodule submodule)
        {
-               d_repository = repository;
                d_submodule = submodule;
-
                d_path = submodule.get_path();
 
+               var repository = submodule.get_owner();
+
                try
                {
                        d_flags = repository.get_submodule_status(submodule.get_name(),
@@ -301,7 +299,7 @@ public class StageStatusEnumerator : Object
 
                                if (!d_ignored_submodules.contains(submodule.get_name()))
                                {
-                                       add(new StageStatusSubmodule(d_repository, submodule));
+                                       add(new StageStatusSubmodule(submodule));
                                }
 
                                return d_cancellable.is_cancelled() ? 1 : 0;


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]