Code review comment for lp:~marcelstimberg/ubuntu/natty/epdfview/fix-for-783109

Revision history for this message
Chris Coulson (chrisccoulson) wrote :

Thanks for your work. However, it seems like you included a few unnecessary files in your patch:

- epdview/THANKS.orig
- epdfview/configure.ac.orig
- epdfview/src/IDocument.cxx.orig
- epdfview/src/PDFDocument.cxx.orig

Also, the diff is huge because the branch is normally pushed with the patches applied, whereas your merge proposal has none applied

review: Needs Fixing

« Back to merge proposal