Merge pull request #25 from programmingkidx/master

Fix bug with adding symbolic links to the wrong object

1 files changed, 5 insertions, 4 deletions

diff --git a/src/Dependency.cpp b/src/Dependency.cpp
index ec37863..1b33f3a 100644
--- a/src/Dependency.cpp
+++ b/src/Dependency.cpp
@@ -193,15 +193,16 @@ std::string Dependency::getInnerPath()
 
 void Dependency::addSymlink(std::string s){ symlinks.push_back(s); }
 
-// comapres the given Dependency with this one. If both refer to the same file,
+// Compares the given Dependency with this one. If both refer to the same file,
 // it returns true and merges both entries into one.
 bool Dependency::mergeIfSameAs(Dependency& dep2)
 {
     if(dep2.getOriginalFileName().compare(filename) == 0)
     {
-        const int samount = dep2.getSymlinkAmount();
-        for(int n=0; n<samount; n++)
-            addSymlink( dep2.getSymlink(n) ); // FIXME - there may be duplicate symlinks
+        const int samount = getSymlinkAmount();
+        for(int n=0; n<samount; n++) {
+            dep2.addSymlink(getSymlink(n)); // FIXME - there may be duplicate symlinks
+        }
         return true;
     }
     return false;