================
@@ -393,7 +392,7 @@ def update_checks_list(clang_tidy_path: str) -> None:
lambda s: os.path.isdir(os.path.join(docs_dir, s)),
os.listdir(docs_dir)
):
for file in filter(
- lambda s: s.endswith(".rst"), os.listdir(os.path.join(docs_dir,
subdir))
+ methodcaller("endswith", ".rst"),
os.listdir(os.path.join(docs_dir, subdir))
----------------
localspook wrote:
In general, I feel the plain loop is easier to read:
```python
for file in os.listdir(os.path.join(docs_dir, subdir):
if file.endswith(".rst"):
oc_files.append((subdir, file))
```
than the current
```python
for file in filter(
methodcaller("endswith", ".rst"), os.listdir(os.path.join(docs_dir, subdir))
):
doc_files.append((subdir, file))
```
https://github.com/llvm/llvm-project/pull/176927
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
