hackerxian / google-diff-match-patch

Automatically exported from code.google.com/p/google-diff-match-patch
Apache License 2.0
0 stars 0 forks source link

cpp version don't compile with gcc-4.4 linux #32

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
pepone@sirius ~/projects/cmsdemo/server $ diff 
/home/pepone/diff_match_patch_20101216/cpp/diff_match_patch.cpp 
diff_match_patch.cpp
22a23
> #include <limits>
70c71
<   qDebug(qPrintable(QString("Diff(") + strOperation(operation) + 
QString(",\"")

---
>   qDebug("%s", qPrintable(QString("Diff(") + strOperation(operation) + 
QString(",\"")

Original issue reported on code.google.com by pepone.o...@gmail.com on 23 Dec 2010 at 9:16

GoogleCodeExporter commented 8 years ago
Thanks.  I don't see the error on any of my platforms, but have made the 
appropriate changes.  That qDebug line should never have been in 
diff_match_patch in the first place, it has been removed.  Similar qDebug lines 
in diff_match_patch_test have been updated as well.

These changes will be in the next push (early next week?)

Original comment by neil.fra...@gmail.com on 28 Dec 2010 at 11:58

GoogleCodeExporter commented 8 years ago
I faced the same problem because I downloaded the code zipfile 
diff_match_patch_20120106.zip, which is marked as "Featured". Maybe it would be 
a good idea to change it or put a notice on the docs. From what I've read, this 
happens because some compilers put the header automatically for you, which is 
not gcc's case.

Original comment by thiago.camposmoraes on 24 May 2012 at 6:25