从 git add -p 编辑大块时修复“补丁不适用”

Question

我对代码文件进行了一系列更改，并在暂存之前执行git add -p path/to/file.hpp 检查差异。

总结一下我的更改：我已将常规类声明转换为模板类定义。在这样做的过程中，我从文件底部取出一大块代码并将其移到顶部（现在需要在类之前定义一个异常类），然后我替换了一堆-liners（方法声明），每个都有几行代码（方法实现）。

Git 无法正确检测到我的更改的上下文，并且基本上已经在一个大杂烩中删除和添加了行，其中两条相邻的行之间根本没有任何联系差异。为了使更改更易于稍后检查，我转移了一堆更改以使它们在上下文中，但小心地保持所有添加和删除的行以相同的顺序，保持添加和删除的行数不变等等.

完成后，我收到错误消息

error: patch failed: include/aof/physics/magnetic-field.hpp:143
error: include/aof/physics/magnetic-field.hpp: patch does not apply
Your edited hunk does not apply. Edit again (saying "no" discards!) [y/n]? 

好的，所以我在某个地方犯了一个错误。好的，我会再试一次。再次发送相同的消息。

如果我在上面回答y，我可以回到我编辑的补丁，但是因为我不知道它有什么问题，所以对我没有多大帮助。在尝试多次编辑补丁失败后，我不禁想知道：有没有办法在这里获得更好的错误消息？我如何找出为什么补丁不适用，所以我可以修复它？

简化示例以阐明我要完成的工作

原始补丁。不太容易看到这里发生了什么......

-        ClassConstructor(const OtherClass& other, double d);
+        ClassConstructor(const TOtherClass& other, double d) : _other(other), _d(d) {

-        void method1() const;
-        double calculation() const;
-        double otherCalculation() const;
+            _a = 1 / d;
+        }

-        ~ClassDestructor() { }; // Yes, of course it's more sensibly named
-     };
+        void method1() const {
+             // this method does nifty stuff.

-    struct my_exception_type : public std::runtime_error {
-        my_execption_type() : runtime_error("oops!") {
         }

-        virtual const char* what() const throw() {
-            std::ostringstream cnvt;
-            cnvt << runtime_error::what() ": Sorry, I shouldn't have done this...";
+        double calculation() const {
+            return _a + _d;
+        }

-            return cnvt.str().c_str();
+        double otherCalculation() const {
+            return 0.; // I'm lazy
         }

+        ~ClassDestructor() { }; // Yes, of course it's more sensibly named
     };

我尝试将其编辑为的内容。 （此编辑在 SO 完成，因此不确定这个特定的是否有问题，但您知道我正在对大块进行什么样的编辑）。更容易理解这些变化，你不觉得吗？

-        ClassConstructor(const OtherClass& other, double d);
+        ClassConstructor(const TOtherClass& other, double d) : _other(other), _d(d) {
+            _a = 1 / d;
+        }

-        void method1() const;
+        void method1() const {
+             // this method does nifty stuff.
+        }

-        double calculation() const;
+        double calculation() const {
+            return _a + _d;
+        }

-        double otherCalculation() const;
+        double otherCalculation() const {
+            return 0.; // I'm lazy
+        }
     };
-    struct my_exception_type : public std::runtime_error {
-        my_execption_type() : runtime_error("oops!") {
-        }
-        virtual const char* what() const throw() {
-            std::ostringstream cnvt;
-            cnvt << runtime_error::what() ": Sorry, I shouldn't have done this...";
-            return cnvt.str().c_str();
-    };

显然，在正确的空行数等方面存在很大的出错风险，但我的问题不仅在于很难确保它全部正确 - 也很难弄清楚我犯了什么错误.

Answer 1

您不能使用git add -p 告诉 git “保留此补丁作为此文件两个版本之间差异的跟踪”：git 确实 不 存储差异或补丁，它存储实际 内容。

当您在 git (git add myfile) 中添加文件时，该文件的特定版本将收到基于其唯一 内容 的哈希值。 Git 将不跟踪它是如何从版本 n-1 到版本 n。

当你运行git diff时，git实际上会检索文件的两个版本的内容，并再次运行diff算法。您尝试编辑的补丁不会保存在内存中。

---

我能想到的最接近的方法是为 diff 算法本身添加“同步点”：

1- 从文件的第一个版本开始，在每个“迁移块”上提交带有 cmets 的第一个中间版本：

// migrating method1 :
void medthod1() const;

//migrating calculation :
void calculation() const;

//etc ...

2- 在第一次提交之后，写下你的实际迁移，保留 cmets：

// migrating method1 :
void method1() const {
    //this method does nifty stuff
}

// migrating calculation :
void calculation() const {
    return _a + _d;
}

//etc ...

并提交此版本。

3- 最后，删除不需要的、嘈杂的 cmets，并提交最后一个版本。

在浏览 git 历史时，迁移将显示为 3 次提交，在检查提交 2- 引起的差异时，您应该清楚地了解重写的内容。