Objectionable settings

[richo]

Well, you did say if anyone objects to anything...

incsearch breaks the "install on remote servers" usecase imo, in the case of a crap connection or a machine swapping violently, vim trying to redraw the screen on each keystroke is a pain. Also, the option has bugged me since day 1, but that's a more scientific objection than simply "it bugs me".

fileformats+=mac I'm assuming this doesn't actually set the line ending to CR like the first docs I found suggest, since even OSX doesn't do this any more. Surely if you're going to pick a horse in this race, unix is the right default?

smartcase I'm still not sure this is necessarily expected behaviour. If I search for a lowercase identifier I expect to only see lowercase matches, that could just be me though. I foresee the usecase for this config being almost exclusively code, and not free text which I generally assumed was the main usecase for smartcase.

[tpope]

I'm boggling at 'incsearch'. That was my go-to example of "stuff everyone agrees on." Also, I've never had a crap connection for which 'incsearch' was the deciding factor in usability (and if I ever win that jackpot, I'll just override it.) I'll keep your input in mind, but it's going to take more than one person to change my mind on this.

Are you sure you read the help for 'fileformats' and not 'fileformat'? All this does is tell Vim to consider that the file might be Mac format after checking for UNIX and DOS line endings.

'smartcase' is relevant only if 'ignorecase' is set. I can't stand 'ignorecase', and this option makes it slightly more tolerable. But I think I'll drop it just because an ineffective setting has the potential to confuse.

[rizzatti]

@tpope leave smartcase, just in case. I think you were right from the beginning on that one. Maybe just a comment in front of it, stating it is in effect only whenever ignorecase is on.

[richo]

Thanks for the feedback. I was reasonably sure that I'm in the minority with finding incsearch incredibly annoying (I have had the connection issue I described, but it's not a regular occurance by any stretch) but thought it was at least putting that vote out there.

You're right about fileformats, I missed the trailing s. As you were.

The help for smartcase is a little confusing, after reading it I thought it implied ignorecase, but I now see that isn't the case. Maybe leaving it in, with a comment describing it's effect and intention would be the right middleground.

[tpope]

@zehrizzatti the other half of this is that I'd rather discourage the use of 'ignorecase' rather than compensate for it. The more 'ignorecase' burns people, the less people will set it. Hopefully.

I have some ideas about other ways to battle 'ignorecase'. But that's for another plugin.

[sedm0784]

I just discovered another person that doesn't like incsearch. So now there are at least two of you!

[richo]

DOZENS OF US