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

Add support for generics syntax in parser #317

Merged
merged 10 commits into from
Jan 22, 2025
Merged

Add support for generics syntax in parser #317

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
0bfeb01
Add support for `::<>` syntax
GuillaumeGomez Jan 9, 2025
960d039
Support recursive generics
GuillaumeGomez Jan 16, 2025
f2e701e
Allow generics on filters
GuillaumeGomez Jan 17, 2025
73bb670
Allow to have generics on function calls
GuillaumeGomez Jan 16, 2025
2969a91
Add more generics tests
GuillaumeGomez Jan 17, 2025
16ce176
parser: count number of `&` in type
Kijewski Jan 20, 2025
89ecbb6
derive: let filters handle generics
Kijewski Jan 20, 2025
4b15a6e
parser: `TyGenerics` is recursive, keep track of nesting level
Kijewski Jan 20, 2025
880be82
parser: parse paths in generic arguments
Kijewski Jan 20, 2025
1cf81ba
Simplify filters handling
GuillaumeGomez Jan 22, 2025
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
2 changes: 1 addition & 1 deletion rinja_derive/src/generator.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -409,7 +409,7 @@ fn is_copyable_within_op(expr: &Expr<'_>, within_op: bool) -> bool {
// The result of a call likely doesn't need to be borrowed,
// as in that case the call is more likely to return a
// reference in the first place then.
Expr::Call(..) | Expr::Path(..) | Expr::Filter(..) | Expr::RustMacro(..) => true,
Expr::Call { .. } | Expr::Path(..) | Expr::Filter(..) | Expr::RustMacro(..) => true,
// If the `expr` is within a `Unary` or `BinOp` then
// an assumption can be made that the operand is copy.
// If not, then the value is moved and adding `.clone()`
Expand Down
Loading
Loading