skip the BOM as well (#301)

Author: Byron

Cargo.lock

@@ -15,6 +15,7 @@ doctest = false
 [dependencies]
 bstr = { version = "0.2.13", default-features = false, features = ["std"]}
 bitflags = "1.3.2"
+unicode-bom = "1.1.4"
 
 [dev-dependencies]
 git-testtools = { path = "../tests/tools"}

git-attributes/Cargo.toml

@@ -8,8 +8,9 @@ pub struct Iter<'a> {
 
 impl<'a> Iter<'a> {
     pub fn new(buf: &'a [u8]) -> Self {
+        let bom = unicode_bom::Bom::from(buf);
         Iter {
-            lines: buf.lines(),
+            lines: buf[bom.len()..].lines(),
             line_no: 0,
         }
     }

git-attributes/src/parse/ignore.rs

@@ -1,6 +1,14 @@
 use git_attributes::ignore::pattern::Mode;
 use git_testtools::fixture_path;
 
+#[test]
+fn byte_order_marks_are_no_patterns() {
+    assert_eq!(
+        git_attributes::parse::ignore("\u{feff}hello".as_bytes()).next(),
+        Some((r"hello".into(), Mode::NO_SUB_DIR, 1))
+    );
+}
+
 #[test]
 fn line_numbers_are_counted_correctly() {
     let ignore = std::fs::read(fixture_path("ignore/various.txt")).unwrap();