From f412b4f96fa49f4ed856db25c10bdf4b9c2e4cec Mon Sep 17 00:00:00 2001
From: w0rp <devw0rp@gmail.com>
Date: Fri, 13 Jan 2017 09:23:03 +0000
Subject: [PATCH] Fix some naming conventions and use abort for all Rust
 functions, and disable the rust linters for now, re #256

---
 README.md                  |  1 -
 ale_linters/rust/cargo.vim |  6 +++---
 ale_linters/rust/rustc.vim | 17 ++++++++++-------
 doc/ale.txt                |  1 -
 4 files changed, 13 insertions(+), 12 deletions(-)

diff --git a/README.md b/README.md
index c65f6cc5..200cc5d9 100644
--- a/README.md
+++ b/README.md
@@ -77,7 +77,6 @@ name. That seems to be the fairest way to arrange this table.
 | Puppet | [puppet](https://puppet.com), [puppet-lint](https://puppet-lint.com) |
 | Python | [flake8](http://flake8.pycqa.org/en/latest/), [pylint](https://www.pylint.org/) |
 | Ruby | [rubocop](https://github.com/bbatsov/rubocop) |
-| Rust | [rustc](https://www.rust-lang.org/), cargo (see `:help ale-integration-rust` for configuration instructions) |
 | SASS | [sass-lint](https://www.npmjs.com/package/sass-lint), [stylelint](https://github.com/stylelint/stylelint) |
 | SCSS | [sass-lint](https://www.npmjs.com/package/sass-lint), [scss-lint](https://github.com/brigade/scss-lint), [stylelint](https://github.com/stylelint/stylelint) |
 | Scala | [scalac](http://scala-lang.org) |
diff --git a/ale_linters/rust/cargo.vim b/ale_linters/rust/cargo.vim
index 0cfc1665..0f04f275 100644
--- a/ale_linters/rust/cargo.vim
+++ b/ale_linters/rust/cargo.vim
@@ -2,7 +2,7 @@
 " Description: rustc invoked by cargo for rust files
 
 
-function! ale_linters#rust#cargo#cargo_or_not_cargo(bufnr)
+function! ale_linters#rust#cargo#GetCargoExecutable(bufnr)
     if ale#util#FindNearestFile(a:bufnr, 'Cargo.toml') !=# ''
         return 'cargo'
     else
@@ -14,8 +14,8 @@ endfunction
 
 call ale#linter#Define('rust', {
 \   'name': 'cargo',
-\   'executable_callback': 'ale_linters#rust#cargo#cargo_or_not_cargo',
+\   'executable_callback': 'ale_linters#rust#cargo#GetCargoExecutable',
 \   'command': 'cargo rustc -- --error-format=json -Z no-trans',
-\   'callback': 'ale_linters#rust#rustc#handle_rustc_errors',
+\   'callback': 'ale_linters#rust#rustc#HandleRustcErrors',
 \   'output_stream': 'stderr',
 \})
diff --git a/ale_linters/rust/rustc.vim b/ale_linters/rust/rustc.vim
index 0c1f3028..b0f82bb2 100644
--- a/ale_linters/rust/rustc.vim
+++ b/ale_linters/rust/rustc.vim
@@ -6,7 +6,10 @@ if !exists('g:ale_rust_ignore_error_codes')
 endif
 
 
-function! ale_linters#rust#rustc#handle_rustc_errors(buffer_number, errorlines)
+function! ale_linters#rust#rustc#HandleRustcErrors(buffer_number, errorlines) abort
+    " FIXME: Fix this linter
+    return []
+
     let l:file_name = fnamemodify(bufname(a:buffer_number), ':t')
     let l:output = []
 
@@ -37,7 +40,7 @@ function! ale_linters#rust#rustc#handle_rustc_errors(buffer_number, errorlines)
             else
                 " when the error is caused in the expansion of a macro, we have
                 " to bury deeper
-                let l:root_cause = s:find_error_in_expansion(l:span, l:file_name)
+                let l:root_cause = s:FindErrorInExpansion(l:span, l:file_name)
 
                 if !empty(l:root_cause)
                     call add(l:output, {
@@ -59,20 +62,20 @@ endfunction
 
 
 " returns: a list [lnum, col] with the location of the error or []
-function! s:find_error_in_expansion(span, file_name)
+function! s:FindErrorInExpansion(span, file_name) abort
     if a:span.file_name ==# a:file_name
         return [a:span.line_start, a:span.byte_start]
     endif
 
     if !empty(a:span.expansion)
-        return s:find_error_in_expansion(a:span.expansion.span, a:file_name)
+        return s:FindErrorInExpansion(a:span.expansion.span, a:file_name)
     endif
 
     return []
 endfunction
 
 
-function! ale_linters#rust#rustc#rustc_command(buffer_number)
+function! ale_linters#rust#rustc#RustcCommand(buffer_number) abort
     " Try to guess the library search path. If the project is managed by cargo,
     " it's usually <project root>/target/debug/deps/ or
     " <project root>/target/release/deps/
@@ -93,7 +96,7 @@ endfunction
 call ale#linter#Define('rust', {
 \   'name': 'rustc',
 \   'executable': 'rustc',
-\   'command_callback': 'ale_linters#rust#rustc#rustc_command',
-\   'callback': 'ale_linters#rust#rustc#handle_rustc_errors',
+\   'command_callback': 'ale_linters#rust#rustc#RustcCommand',
+\   'callback': 'ale_linters#rust#rustc#HandleRustcErrors',
 \   'output_stream': 'stderr',
 \})
diff --git a/doc/ale.txt b/doc/ale.txt
index 689b7293..f0c5b162 100644
--- a/doc/ale.txt
+++ b/doc/ale.txt
@@ -91,7 +91,6 @@ The following languages and tools are supported.
 * Pug: 'pug-lint'
 * Puppet: 'puppet', 'puppet-lint'
 * Python: 'flake8', 'pylint'
-* Rust: 'rustc' (see |ale-integration-rust|)
 * Ruby: 'rubocop'
 * SASS: 'sasslint', 'stylelint'
 * SCSS: 'sasslint', 'scsslint', 'stylelint'