Hi, 2016/1/31 Sun 0:02:07 UTC+9 Ken Takata wrote: > Hi, > > 2016/1/30 Sat 12:51:11 UTC+9 Ken Takata wrote: > > Hi, > > > > 2016/1/21 Thu 7:11:51 UTC+9 Bram Moolenaar wrote: > > > Ken Takata wrote: > > > > > > > 2016/1/20 Wed 7:47:18 UTC+9 Ken Takata wrote: > > > > > Hi Bram, > > > > > > > > > > 2016/1/20 Wed 3:33:18 UTC+9 Bram Moolenaar wrote: > > > > > > Ken Takata wrote: > > > > > > > > > > > > > (The original thread was in vim_use: > > > > > > > https://groups.google.com/d/topic/vim_use/RLM8Vqa_aME/discussion ) > > > > > > > > > > > > > > I have updated the patches for supporting large files on Windows. > > > > > > > I also converted the test to the new style. And I also added a > > > > > > > new test for > > > > > > > large files: test_largefile.vim. This creates a 4GB file, moving > > > > > > > around it, > > > > > > > and write it to disk. This uses over 8GB disk spaces, and > > > > > > > consumes CPU > > > > > > > powers. So I don't add this test to src/testdir/Make_all.mak. I > > > > > > > confirmed > > > > > > > that when running the test without my fix, gvim.exe (32-bit) > > > > > > > stopped with > > > > > > > "E342: Out of memory!" (takes about 10min), but it succeeded with > > > > > > > my fix > > > > > > > (takes about 5min). > > > > > > > Please check the attached patches. > > > > > > > > > > > > > > Note: Please apply the following patch before them to avoid > > > > > > > conflicts. > > > > > > > https://groups.google.com/d/topic/vim_dev/JHbE4twU4dk/discussion > > > > > > > > > > > > Thanks. In what order do the patches need to be applied? > > > > > > Looks like the stat_T patch comes after the others? > > > > > > > > > > Yes. > > > > > use-stat_T.patch must be applied after > > > > > support-largefiles-on-windows.patch. > > > > > Other two are independent. > > > > > > > > > > > > > > > > For the test, we can add a "torture" target. > > > > > > > > > > > > I have this old note in the todo file: > > > > > > > > > > > > Win32: patch to use 64 bit stat() if possible. (Ken Takata, 2014 > > > > > > May 12) > > > > > > More tests May 14. Update May 29. Update Aug 10. > > > > > > > > > > > > I assume that is related to this? > > > > > > > > > > Yes, the stat_T patch is a patch for this. > > > > > Without this patch, getfsize() returns a broken value (e.g. > > > > > -294967296), if > > > > > the size of a file is larger than 2 GiB and smaller than 4 GiB. And > > > > > if the > > > > > size is larger than 4 GiB, getfsize() and other stat functions return > > > > > -1. > > > > > MSVC's 32-bit stat() always returns error if the filesize is larger > > > > > than 4 GiB. > > > > > > > > I wrote an additional patch to fix display of off_T value. > > > > add-stat-test.patch is updated to fix conflicts with 7.4.1142. > > > > Please apply in the following order: > > > > > > > > 1. support-largefiles-on-windows.patch (previous mail) > > > > 2. fix-off_T-display.patch (this mail) > > > > 3. use-stat_T.patch (previous mail) > > > > 4. add-stat-test.patch (this mail) > > > > 5. test_largefile.patch (previous mail) > > > > > > Thanks! > > > > Here are the updated large files patch for 7.4.1202. > > And... this is for 7.4.1205.
Oops, I sent my update too early. I have to wait all the style changes have completed. Regards, Ken Takata -- -- You received this message from the "vim_dev" maillist. Do not top-post! Type your reply below the text you are replying to. For more information, visit http://www.vim.org/maillist.php --- You received this message because you are subscribed to the Google Groups "vim_dev" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. For more options, visit https://groups.google.com/d/optout.
