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!
--
hundred-and-one symptoms of being an internet addict:
25. You believe nothing looks sexier than a man in boxer shorts illuminated
only by a 17" inch svga monitor.
/// Bram Moolenaar -- [email protected] -- http://www.Moolenaar.net \\\
/// sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
\\\ an exciting new programming language -- http://www.Zimbu.org ///
\\\ help me help AIDS victims -- http://ICCF-Holland.org ///
--
--
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.
