{"push_id":573695006,"size":32,"distinct_size":1,"ref":"refs/heads/experimentZip","head":"302d5412dddb755d47552c905eb9c6122a7cf3c0","before":"e8d93d482ae0c45a65cbe5b1f9dfe3e49646f640","commits":[{"sha":"13846644c85b27a4500717ddb2e02e07e56fa041","author":{"email":"ca73de655f9b87b054053fc0c2c530ceaa35eb37@apache.org","name":"Dennis Hamilton"},"message":"extract_downloads.bat and helper unzip-win.js 1.00\n\nThe pair of scripts provide custom extraction of external downloads for\nwin32 x86 dependencies from Corinthia components.   Implementation uses\nnative Windows technology with no additional components required.","distinct":false,"url":"https://api.github.com/repos/apache/incubator-corinthia/commits/13846644c85b27a4500717ddb2e02e07e56fa041"},{"sha":"a48521515f567038768287d67448cb5748ae3df3","author":{"email":"ca73de655f9b87b054053fc0c2c530ceaa35eb37@apache.org","name":"Dennis Hamilton"},"message":"Completed  fetch_downloads extract_downloads pairing\n\nThe procedures work together and extract_downloads always calls\nfetch_downloads just in case needed (already-downloaded files are not\nrefetched).  Basic tests are successful.","distinct":false,"url":"https://api.github.com/repos/apache/incubator-corinthia/commits/a48521515f567038768287d67448cb5748ae3df3"},{"sha":"2d3745056c41dd0d45484b07580b97713e1dcf52","author":{"email":"ca73de655f9b87b054053fc0c2c530ceaa35eb37@apache.org","name":"Dennis Hamilton"},"message":"Add .gitattributes\n\nThe Git attributes are set to adjust line endings between different\nclients and the repository form (usually Unix NL instead of CR-LF).","distinct":false,"url":"https://api.github.com/repos/apache/incubator-corinthia/commits/2d3745056c41dd0d45484b07580b97713e1dcf52"},{"sha":"944312a8cfa04c5ae55f28fee529097eda4619f4","author":{"email":"ca73de655f9b87b054053fc0c2c530ceaa35eb37@apache.org","name":"Dennis Hamilton"},"message":".gitattributes 1.1.0\n\nAn inventory of known test documents is reviewed and the various\nbinary-format extensions added to the .gitattributes, now versioned and\ndated so that there are clues about what is current.","distinct":false,"url":"https://api.github.com/repos/apache/incubator-corinthia/commits/944312a8cfa04c5ae55f28fee529097eda4619f4"},{"sha":"40fb232e4fa18eb7eb5c09ca9661a5508c7e2fe5","author":{"email":"ca73de655f9b87b054053fc0c2c530ceaa35eb37@apache.org","name":"Dennis Hamilton"},"message":"external\\README.txt replacement draft\n\nA 1.0 Draft of the README is provided in place of the previous one that\nreferenced the bash scripts.  This attempts to cover the essentials with\ndrill-down into maintenance and customization to be in separate files.","distinct":false,"url":"https://api.github.com/repos/apache/incubator-corinthia/commits/40fb232e4fa18eb7eb5c09ca9661a5508c7e2fe5"},{"sha":"2d963742473b25462502b491a1bd67d6b2097f66","author":{"email":"ca73de655f9b87b054053fc0c2c530ceaa35eb37@apache.org","name":"Dennis Hamilton"},"message":"Completed External Downloading Description\n\nAdditional description files are provided to guide the running and the\nmaintenance of the Windows procedures.","distinct":false,"url":"https://api.github.com/repos/apache/incubator-corinthia/commits/2d963742473b25462502b491a1bd67d6b2097f66"},{"sha":"f058c02a0100dc16f868745594788e96e1c84282","author":{"email":"ca73de655f9b87b054053fc0c2c530ceaa35eb37@apache.org","name":"Dennis Hamilton"},"message":"Change location of external extractions\n\nThe extract_downloads.bat script used folders within external/download\nfor include, lib, and bin.  The simplest correction was to change\nCMakeLists.txt to agree rather than make many more fixes to\nextract_downloads.bat.","distinct":false,"url":"https://api.github.com/repos/apache/incubator-corinthia/commits/f058c02a0100dc16f868745594788e96e1c84282"},{"sha":"3f11aefa27314b66eee44b14561cf3514a061537","author":{"email":"4b8373d016f277527198385ba72fda0feb5da015@uxproductivity.com","name":"Peter Kelly"},"message":"Don't include <io.h> on non-Windows platforms\n\nThe <io.h> system header is only avilable on windows. It was\nunconditionally included in DFFilesystem.c, which broke the build on\nLinux and OS X. It used to be wrapped in an #ifdef WIN32 but with the\nremoval of this, it was already included.\n\nI have now moved the #include <io.h> to DFPlatform.h, where there is an\nifdef in place to distinguish between WIN32 and other platforms, so that\nwe don't get a compile error when this is built on Linux and OS X.","distinct":false,"url":"https://api.github.com/repos/apache/incubator-corinthia/commits/3f11aefa27314b66eee44b14561cf3514a061537"},{"sha":"cc208a6d44534fea95a5fa2f4bcc7f733b58fd89","author":{"email":"4b8373d016f277527198385ba72fda0feb5da015@uxproductivity.com","name":"Peter Kelly"},"message":"Fix OS X build breakage in DFError.h\n\nCommit e8d93d4 removed the ATTRIBUTE_FORMAT macro from DFError.h, since\nthis macro is also present in DFPlatform.h. However, DFError.h is part\nof the public API, while DFPlatform is not. Consumers that are\nconfigured to compile only against the public API are unable to access\nmacro definitions in DFPlatform.h, so those macros must be present in\nthe API headers.\n\nThe ATTRIBUTE_FORMAT is used for telling the compiler to check the\narguments to DFErrorFormat to make sure they conform to printf-style\nformatting. GCC/Clang and Visual C++ both have a different syntax for\nthis, which is why a macro is needed.\n\nI'm not sure why gcc on Linux and Visual C++ on windows weren't\ncomplaining about this, but Xcode gave a compile error as the macro was\nundefined.\n\nIf we want to avoid platform-specific #ifdefs in specific public header\nfiles, we'll need something equivalent to DFPlatform.h which is included\nin the public headers. Putting this #ifdef back as-is is a temporary\nsolution, needed to build the code under Xcode, but ultimately it would\nbe good if we could come up with a more general solution.","distinct":false,"url":"https://api.github.com/repos/apache/incubator-corinthia/commits/cc208a6d44534fea95a5fa2f4bcc7f733b58fd89"},{"sha":"cd6ccd0e7625384733db58d92dca7a87249d1247","author":{"email":"4b8373d016f277527198385ba72fda0feb5da015@uxproductivity.com","name":"Peter Kelly"},"message":"dfutil: Fix pretty-printing option (-pp)\n\nWord_toPlainFromDir was incorrectly interpreting relative path names in\nOPC relationship files for word documents. Instead of calculating the\npath relative to the root of the package, it would use the path relative\nto the location of the document.xml file.\n\nThis was causing the pretty-printing routine to encounter an error when\nlooking for files like styles.xml, because it would check for them in\nthe root of the package, rather than the directory in which they were\nlocated (usually /word).","distinct":false,"url":"https://api.github.com/repos/apache/incubator-corinthia/commits/cd6ccd0e7625384733db58d92dca7a87249d1247"},{"sha":"22947efbb19452866cf838ca114341253be03011","author":{"email":"4b8373d016f277527198385ba72fda0feb5da015@uxproductivity.com","name":"Peter Kelly"},"message":"Fix build on windows (VC++ parser error)\n\nForgot to add a ;; required to get around a bug in the Visual C++ parser\nin the case where a single-line if body immediately precedes a line\ncontaining a variable declaration with a typedef'd type.","distinct":false,"url":"https://api.github.com/repos/apache/incubator-corinthia/commits/22947efbb19452866cf838ca114341253be03011"},{"sha":"5204f74e892e98e5b877a79946b208a2279eed5c","author":{"email":"4b8373d016f277527198385ba72fda0feb5da015@uxproductivity.com","name":"Peter Kelly"},"message":"Get rid of DFCreateTempDir\n\nFor a number of operations, like running tests and converting documents,\nwe used to require the existence of a temporary directory into which we\ncould extract the contents of a .docx or other zip-based package.\nHowever, since the implementation of the DFStorage interface, with both\nan in-memory implementation, and a zip-based implementation, it's no\nlonger necessary to write the XML files from a zip package out to the\nfilesystem.\n\nDFCreateTempDir used to rely on mkdtemp, which generates temporary\ndirectories in a secure manner (that is, avoiding a race condition\nbetween the directory's creation time and the application getting\nexclusive access to it). However, this function is not available on\nwindows, leaving us with the choice of having an #ifdef WIN32 to use a\nfixed directory name on windows, or doing the same thing on all\nplatforms. Compiling on Linux would give a warning about the mktemp\nfunction being unsafe.\n\nGiven that we don't actually need temporary directories any more for\ntesting or conversion though, we can just remove this function\naltogethter, avoiding this dilemma.\n\nThe one place where temporary files are still used is when dftest is run\nwith the -diff option, which prints out the differences between the\nexpected and actual results of a test case if there is a failure. This\nhas been changed to use the fixed filenames 'dftest-diff-from.tmp' and\n'dftest-diff-to.tmp' so it can run the diff command as a sub-process.\n\nAnother nice advantage of this is when dftest or dfutil crashes, you\ndon't get a dfutil.XXXXXX directory lying around.","distinct":false,"url":"https://api.github.com/repos/apache/incubator-corinthia/commits/5204f74e892e98e5b877a79946b208a2279eed5c"},{"sha":"db23077f5ecc6cebdf2ab4b2e01242fd6dc5e6bb","author":{"email":"4b8373d016f277527198385ba72fda0feb5da015@uxproductivity.com","name":"Peter Kelly"},"message":"Shut git up about CRLF issues","distinct":false,"url":"https://api.github.com/repos/apache/incubator-corinthia/commits/db23077f5ecc6cebdf2ab4b2e01242fd6dc5e6bb"},{"sha":"474eb4caeb29f2353a0fadeef3f5a6bd8bc3b5ee","author":{"email":"4b8373d016f277527198385ba72fda0feb5da015@uxproductivity.com","name":"Peter Kelly"},"message":"Word: Make conversion function names consistent\n\nWordConverterUpdateFromHTML -> WordConverterPut\nWordConverterConvertToHTML  -> WordConverterGet\nWordPackageUpdateFromHTML   -> WordPackagePut\nWordPackageGenerateHTML     -> WordPackageGet","distinct":false,"url":"https://api.github.com/repos/apache/incubator-corinthia/commits/474eb4caeb29f2353a0fadeef3f5a6bd8bc3b5ee"},{"sha":"ce0e3d58d5602f2d63a9c8d325451180108fd980","author":{"email":"4b8373d016f277527198385ba72fda0feb5da015@uxproductivity.com","name":"Peter Kelly"},"message":"Word: Conversion functions - simpler organisation\n\nInstead of having WordPackageGet and WordPackagePut (which simply\ncreated a WordConverter object and then called through to the relevant\nget or put method), do this directly in the WordGet/WordPut functions.\n\nAlso move some functionality into the WordConverter get/put functions\nthat really belongs there, rather than outside.\n\nUsers of WordConverter no longer have to create an instance of the class\nfirst; they simply call WordConverterGet or WordConverterPut. These\nfunctions create an instance internally, simplifying their use. The\nparameter both of these functions take are what was previosuly passed to\nWordConverterNew.","distinct":false,"url":"https://api.github.com/repos/apache/incubator-corinthia/commits/ce0e3d58d5602f2d63a9c8d325451180108fd980"},{"sha":"d785d58d96bc807c7546f683a860b5a8c7c6d3aa","author":{"email":"4b8373d016f277527198385ba72fda0feb5da015@uxproductivity.com","name":"Peter Kelly"},"message":"Remove <io.h> include\n\nThis was needed at one point, but I can't remember what for. It's no\nlonger required; the code builds fine without it.","distinct":false,"url":"https://api.github.com/repos/apache/incubator-corinthia/commits/d785d58d96bc807c7546f683a860b5a8c7c6d3aa"},{"sha":"5c9a2c08ea370cf02ea1d76b4a1800ea8076e64b","author":{"email":"4b8373d016f277527198385ba72fda0feb5da015@uxproductivity.com","name":"Peter Kelly"},"message":"dfutil: Fix zip/unzip parameter order in help text","distinct":false,"url":"https://api.github.com/repos/apache/incubator-corinthia/commits/5c9a2c08ea370cf02ea1d76b4a1800ea8076e64b"},{"sha":"bc6827f6130a8b04b3c97598c31d4cc7289a2ddb","author":{"email":"4b8373d016f277527198385ba72fda0feb5da015@uxproductivity.com","name":"Peter Kelly"},"message":"Word: Remove warnings parameters for conversion\n\nWe've always treated warnings as errors, so we might as well disperse\nwith the idea of exposing a separate data structure to code that uses\nWordConverter to record these.\n\nThe warnings buffer still exists for now; it's just internal to\nWordConverter.","distinct":false,"url":"https://api.github.com/repos/apache/incubator-corinthia/commits/bc6827f6130a8b04b3c97598c31d4cc7289a2ddb"},{"sha":"1706df12c1de93580e427a409a79f669c0e71c99","author":{"email":"4b8373d016f277527198385ba72fda0feb5da015@uxproductivity.com","name":"Peter Kelly"},"message":"Word: Move WordConverter{Get,Put}2 code\n\nPreviously, the functions WordConverterGet and WordConverterPut would\nsimply create a WordConverter object and then call through to the '2'\nversion of the same function.\n\nGiven that there's little in the way of error handling that needs to\nhappen in the latter, it makes sense to do it all in one function.","distinct":false,"url":"https://api.github.com/repos/apache/incubator-corinthia/commits/1706df12c1de93580e427a409a79f669c0e71c99"},{"sha":"2230024704d9acda7b4a705e6b9da9e0c6304803","author":{"email":"4b8373d016f277527198385ba72fda0feb5da015@uxproductivity.com","name":"Peter Kelly"},"message":"Word: Remove idPrefix parameter\n\nIt used to be that code using the DocFormats library would supply a\nprefix to be used on all id attributes in the HTML file which were used\nto maintain a mapping back to the elements in the original document.\nThe reason for this was that in UX Write, if you copied & pasted from\none word document to another, the HTML code would be pasted as-is, and\nthis could result in incorrect mappings being stored in the destination\nfile.\n\nUX Write handles this differently now (can't remember the details; it\nwas a long time ago I made the change), and for a long time we've been\nusing the fixed prefix of \"word\" for all id attributes. So we don't need\nthat as a parameter to the conversion functions.","distinct":false,"url":"https://api.github.com/repos/apache/incubator-corinthia/commits/2230024704d9acda7b4a705e6b9da9e0c6304803"}]}