2017-04-01 11:19 GMT+09:00 Kazunobu Kuriyama <[email protected]>:

> 2017-04-01 11:11 GMT+09:00 Kazunobu Kuriyama <[email protected]>
> :
>
>> 2017-04-01 6:33 GMT+09:00 Bram Moolenaar <[email protected]>:
>>
>>>
>>> Patch 8.0.0531 (after 8.0.0530)
>>> Problem:    Test with long directory name fails on non-unix systems.
>>> Solution:   Skip the test on non-unix systems.
>>> Files:      src/testdir/test_edit.vim
>>>
>>>
>>> *** ../vim-8.0.0530/src/testdir/test_edit.vim   2017-03-31
>>> 22:27:08.617328400 +0200
>>> --- src/testdir/test_edit.vim   2017-03-31 23:29:29.722486313 +0200
>>> ***************
>>> *** 1324,1329 ****
>>> --- 1324,1333 ----
>>>   endfunc
>>>
>>>   func Test_edit_complete_very_long_name()
>>> +   if !has('unix')
>>> +     " Long directory names only work on Unix.
>>> +     return
>>> +   endif
>>>     let save_columns = &columns
>>>     set columns=5000
>>>     call assert_equal(5000, &columns)
>>> *** ../vim-8.0.0530/src/version.c       2017-03-31 22:27:08.621328374
>>> +0200
>>> --- src/version.c       2017-03-31 23:32:01.033557892 +0200
>>> ***************
>>> *** 766,767 ****
>>> --- 766,769 ----
>>>   {   /* Add new patch number below this line */
>>> + /**/
>>> +     531,
>>>   /**/
>>>
>>> --
>>> "Marriage is the process of finding out what kind of man your wife
>>> would have preferred"
>>>
>>
>>
>> According to Travis #3022.2 and #3022.4, the new test still fails on
>> macOS.
>>
>> That is due to that the OS is a little bit picky about the length of each
>> path component as well as the whole length of a path name.
>>
>> Hopefully, the attached patch fixes the failures.
>>
>> Best regards,
>> Kazunobu Kuriyama
>>
>
I was confused.   Since macOS's default file system, HFS Plus, doesn't
reserve slash (/), I thought slashes in a path had to be counted into the
length computations.  But I found that the replacements of 255 with 254
were wrong.  The attached patch corrects that (I wish...)


>
>>
>>
>>>  /// 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.
>>>
>>
>>
>

-- 
-- 
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.

Attachment: fix-test_edit-for-macos-update02.patch
Description: Binary data

Raspunde prin e-mail lui