Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Parallel formatting to increase speed #6095

Open
wants to merge 23 commits into
base: master
Choose a base branch
from
Open

Parallel formatting to increase speed #6095

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
23 commits
Select commit Hold shift click to select a range
3aed0b7
Can run multithreaded
MarcusGrass Feb 23, 2024
9e28f78
Use a map to collect thread panics
MarcusGrass Feb 23, 2024
75151cc
Pass printer down the stack
MarcusGrass Feb 25, 2024
7456b97
Compiles
MarcusGrass Feb 25, 2024
378e106
Rip out auto term-printing
MarcusGrass Feb 25, 2024
4496f28
Emitters dont print
MarcusGrass Feb 25, 2024
ae322ed
Buggy works
MarcusGrass Feb 25, 2024
adef47a
Fixup most tests
MarcusGrass Feb 25, 2024
606d89e
Formatted passes tests
MarcusGrass Feb 25, 2024
1c3f955
Pre dump
MarcusGrass Feb 25, 2024
ef30168
Fixup `rustc_error` print output
MarcusGrass Feb 26, 2024
70cc3b5
Fixup stdin echo, no newline add
MarcusGrass Feb 26, 2024
b1e1cd4
Fmt
MarcusGrass Feb 26, 2024
b749673
Use available parallelism
MarcusGrass Feb 26, 2024
7297078
break out join handling
MarcusGrass Feb 26, 2024
2d8c7aa
Add some comments
MarcusGrass Feb 26, 2024
e10204d
Remove printer from emitter trait
MarcusGrass Feb 26, 2024
81fa030
Comments, reorder, rename
MarcusGrass Feb 26, 2024
eb78a34
Clean up a bit more
MarcusGrass Feb 26, 2024
bdf8291
Write more on progress
MarcusGrass Feb 26, 2024
9505f43
Leave attrs.rs, move merge md into PR
MarcusGrass Feb 26, 2024
1928408
Clean up imports
MarcusGrass Feb 26, 2024
de08ccd
Comment, import, and minor code cleanup
MarcusGrass Feb 26, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Leave attrs.rs, move merge md into PR
  • Loading branch information
@MarcusGrass
MarcusGrass committed Feb 26, 2024
commit 9505f435a4f07427576dc8a1475f4f62ae2d31a5
66 changes: 0 additions & 66 deletions Progress.md
View file
Open in desktop

This file was deleted.

6 changes: 1 addition & 5 deletions config_proc_macro/src/attrs.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -68,11 68,7 @@ fn get_name_value_str_lit(attr: &syn::Attribute, name: &str) -> Option<String> {
match &attr.meta {
syn::Meta::NameValue(syn::MetaNameValue {
path,
value:
syn::Expr::Lit(syn::ExprLit {
lit: syn::Lit::Str(lit_str),
..
}),
value: syn::Expr::Lit(syn::ExprLit { lit: syn::Lit::Str(lit_str), .. }),
..
}) if path.is_ident(name) => Some(lit_str.value()),
_ => None,
Expand Down