Skip to content

Commit 1242151

Browse files
ByronEliahKagan
andcommitted
Apply suggestions from code review
Co-authored-by: Eliah Kagan <[email protected]>
1 parent d2ae9d5 commit 1242151

File tree

3 files changed

+8
-8
lines changed

3 files changed

+8
-8
lines changed

gix-fs/tests/stack/mod.rs

+2-2
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ fn path_join_handling() {
5959
assert_eq!(
6060
p("c:").join("relative"),
6161
p("c:relative"),
62-
"absolute + relative = strange joined result with missing backslash, but it's a valid path that works just like `c:\relative`"
62+
"drive + relative = strange joined result with missing backslash, but it's a valid path that works just like `c:\relative`"
6363
);
6464
assert_eq!(
6565
p("c:\\").join("relative"),
@@ -102,7 +102,7 @@ fn path_join_handling() {
102102
assert_eq!(
103103
p("c:\\").join("\\\\.\\"),
104104
p("\\\\.\\"),
105-
"d-drive-with-bs + device-namespace-unc = device-namespace-unc"
105+
"c-drive-with-bs + device-namespace-unc = device-namespace-unc"
106106
);
107107
assert_eq!(
108108
p("/").join("C:/"),

gix-ref/src/store/file/mod.rs

+1-1
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ pub struct Store {
2727
pub write_reflog: WriteReflog,
2828
/// The namespace to use for edits and reads
2929
pub namespace: Option<Namespace>,
30-
/// This is only useful on Windows, which may have 'virtual' devices on each level of a path so that
30+
/// This is only needed on Windows, where some device names are reserved at any level of a path, so that
3131
/// reading or writing `refs/heads/CON` for example would read from the console, or write to it.
3232
pub prohibit_windows_device_names: bool,
3333
/// If set, we will convert decomposed unicode like `a\u308` into precomposed unicode like `ä` when reading

gix-validate/tests/path/mod.rs

+5-5
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,7 @@ mod component {
6666
mktest!(dot_gitmodules_as_file, b".gitmodules", UNIX_OPTS);
6767
mktest!(
6868
starts_with_dot_git_with_backslashes_on_linux,
69-
b".git\\hooks\\precommit",
69+
b".git\\hooks\\pre-commit",
7070
UNIX_OPTS
7171
);
7272
mktest!(not_dot_git_shorter, b".gi", NO_OPTS);
@@ -136,7 +136,7 @@ mod component {
136136
mktest!(dot_git_upper, b".GIT", Error::DotGitDir, NO_OPTS);
137137
mktest!(
138138
starts_with_dot_git_with_backslashes_on_windows,
139-
b".git\\hooks\\precommit",
139+
b".git\\hooks\\pre-commit",
140140
Error::PathSeparator
141141
);
142142
mktest!(dot_git_upper_hfs, ".GIT\u{200e}".as_bytes(), Error::DotGitDir);
@@ -199,12 +199,12 @@ mod component {
199199
Error::WindowsIllegalCharacter
200200
);
201201
mktest!(
202-
ntfs_stream_default_explicit,
203-
b"file:$ANYTHING_REALLY:$DATA",
202+
ntfs_stream_explicit,
203+
b"file:ANYTHING_REALLY:$DATA",
204204
Error::WindowsIllegalCharacter
205205
);
206206
mktest!(
207-
dot_gitmodules_lower_ntfs_stream_default_explicit,
207+
dot_gitmodules_lower_ntfs_stream,
208208
b".gitmodules:$DATA:$DATA",
209209
Error::SymlinkedGitModules,
210210
Symlink,

0 commit comments

Comments
 (0)