diff options
| author | Moritz Poldrack <git@moritz.sh> | 2022-07-31 14:32:48 +0200 |
|---|---|---|
| committer | Robin Jarry <robin@jarry.cc> | 2022-08-04 21:58:01 +0200 |
| commit | 978d35d356e8752bdd272884df48a6289d88b40a (patch) | |
| tree | 3910243e688ef503159d07ce44b22cfea5d6c6fd /worker/maildir/container.go | |
| parent | c882cf9960be691fe55617b87cdfcfbabd5d5557 (diff) | |
| download | aerc-978d35d356e8752bdd272884df48a6289d88b40a.tar.gz | |
lint: homogenize operations and minor fixes (gocritic)
Apply GoDoc comment policy (comments for humans should have a space
after the //; machine-readable comments shouldn't)
Use strings.ReplaceAll instead of strings.Replace when appropriate
Remove if/else chains by replacing them with switches
Use short assignment/increment notation
Replace single case switches with if statements
Combine else and if when appropriate
Signed-off-by: Moritz Poldrack <moritz@poldrack.dev>
Acked-by: Robin Jarry <robin@jarry.cc>
Diffstat (limited to 'worker/maildir/container.go')
| -rw-r--r-- | worker/maildir/container.go | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/worker/maildir/container.go b/worker/maildir/container.go index 1d971a4d..35125775 100644 --- a/worker/maildir/container.go +++ b/worker/maildir/container.go @@ -80,7 +80,7 @@ func (c *Container) ListFolders() ([]string, error) { return filepath.SkipDir } dirPath = strings.TrimPrefix(dirPath, ".") - dirPath = strings.Replace(dirPath, ".", "/", -1) + dirPath = strings.ReplaceAll(dirPath, ".", "/") folders = append(folders, dirPath) // Since all mailboxes are stored in a single directory, don't @@ -124,7 +124,7 @@ func (c *Container) Dir(name string) maildir.Dir { if name == "INBOX" { return maildir.Dir(c.dir) } - return maildir.Dir(filepath.Join(c.dir, "."+strings.Replace(name, "/", ".", -1))) + return maildir.Dir(filepath.Join(c.dir, "."+strings.ReplaceAll(name, "/", "."))) } return maildir.Dir(filepath.Join(c.dir, name)) } |