Skip to content

add a parameter for identifying the applied state #13

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
phschoen wants to merge 4 commits into
base: master
Choose a base branch
from
Open

add a parameter for identifying the applied state #13

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
e591001
add a parameter for identifying the applied state
Oct 27, 2016
ee3ae94
add parameter to check if the style is allied
Nov 30, 2016
42c3989
update readme for b:linuxsty_applied and b:apply_linux_style
Nov 30, 2016
eddccf8
correct code quotation marks and remove tailing whitespaces
Nov 30, 2016
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 4 additions & 4 deletions plugin/linuxsty.vim
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,21 +28,21 @@ augroup linuxsty
augroup END

function s:LinuxConfigure()
let apply_style = 0
let b:apply_linux_style = 0

if exists("g:linuxsty_patterns")
let path = expand('%:p')
for p in g:linuxsty_patterns
if path =~ p
let apply_style = 1
let b:apply_linux_style = 1
break
endif
endfor
else
let apply_style = 1
let b:apply_linux_style = 1
endif

if apply_style
if b:apply_linux_style
Copy link
Owner

@vivien vivien Nov 28, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In fact, may I ask you to add a single line let b:linuxsty_applied = 1 in this statement? So that we change only one line and keep consistent with an implicit linuxsty namespace here.

A small addition in the README file to mention this new buffer variable would be a must.
If you tell me that it works as you expect, I'll happily merge this. Thanks!

Copy link
Author

@phschoen phschoen Aug 24, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I added extra commit for the readme file and changed it to b:linuxsty_applied
I did see no problem popping up and its working fine.

call s:LinuxCodingStyle()
endif
endfunction
Expand Down