| Commit message (Collapse) | Author | Age | Files | Lines |
| ... | |
| | |
| |
| |
| | |
The API is a bit clumsy, but we're mostly going to use it internally.
|
| | | |
|
| | |
| |
| |
| |
| | |
Add underscores so that the spec keywords and the flag constants
correspond (according to `paste`'s case-changing rules).
|
| | |
| |
| |
| |
| |
| |
| | |
* Document the bizarre FromStr impl. That isn't in my way, so I
don't intend to fix it. So I've documented it instead.
* Mention the naming discrepancy.
|
| | |
| |
| |
| |
| | |
We're going to do a lot more with this and it's less confusing to have
it all together.
|
| |\ \
| | |
| | |
| | |
| | | |
tor-netdoc: parse2 More deduplication of the derive macro
See merge request tpo/core/arti!3455
|
| | | |
| | |
| | |
| | |
| | | |
This can result in "trait not implemented" errors here. Use the
`${paste_spanned}` trick.
|
| | | | |
|
| | | | |
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Move what is now IF_NONSTRUCTURAL_ACCUMULATE into the common module.
It's not yet suitable for reuse.
The docs for ACCUMULATE_ITEM_VALUE are still wrong. We'll fix that in
a moment.
|
| | | |
| | |
| | |
| | |
| | |
| | | |
This was previously only not done because of the need to c&p.
Also, we're going to be expanding some common macros that will use it.
|
| | | |
| | |
| | |
| | | |
The hygiene is fixed in derive-deftly 1.5.x.
|
| | | | |
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | | |
We're going to introduce facilities for encoding more generally, and
we'll want a way to enable them.
Instead, have a separate feature for *encoding* documents.
|
| |/ /
| |
| |
| |
| | |
The word "build" so often in Rust means builder pattern. This module
contains functions for encoding.
|
| |\ \
| | |
| | |
| | |
| | | |
tor-netdoc: parse2 derive: Some deduplication
See merge request tpo/core/arti!3450
|
| | | | |
|
| | | | |
|
| | | | |
|
| | | | |
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | | |
Have the module expect the use site to determine (mostly) the field
kind. Document this expectation.
Now NetdocParseableCommon can have the keyword-related definitions.
|
| | | |
| | |
| | |
| | | |
Put one textually-identical define in it, to start with.
|
| |/ /
| |
| |
| |
| |
| |
| | |
This has the meta attributes with optional values feature and also
hygiene rework for modules.
The breaking changes don't break arti.
|
| |\ \
| | |
| | |
| | |
| | | |
Bump derive-deftly to 1.4.0
See merge request tpo/core/arti!3448
|
| | |/
| |
| |
| |
| |
| | |
This will let us use the new modules feature.
There are no breaking changes to beta features in 1.4.0.
|
| | |
| |
| |
| | |
Run maint/add_warning
|
| |/ |
|
| | |
|
| |
|
|
|
|
|
| |
```bash
readarray -t publish < <(cargo metadata --format-version 1 | jq -r '.packages[] | select((.id | startswith("path+file:///")) and (.rust_version != null) and (.publish == null or .publish == true or .publish != [])) | .name')
for package in "${publish[@]}"; do echo "$package:"; cargo set-version --bump minor -p "$package"; done
```
|
| |
|
|
| |
Change our mind about the logic for encoding an Ignored.
|
| | |
|
| | |
|
| | |
|
| | |
|
| |
|
|
|
|
|
|
|
| |
Currently this is mishandled: the current use sites (in
rs/each_variety.rs) use Option<Ignored>, which would reject repeated
items with that keyword, which is technically wrong. And encoing
will become a problem. Vec<Ignored> would be wrong too.
Instead, broaden NotPresent and give it multiplicity impls.
|
| |
|
|
|
|
| |
This seems to be where we're putting most of the parse2 impls on these
types, now. That makes more sense as parse2 becomes more of a proper
thing.
|
| | |
|
| | |
|
| |
|
|
|
|
|
|
|
| |
This allows a NetdocParseableFields inside a NetdocParseableFields.
The code is rather duplicative with the NetdocParseable
implementation. This is due to limitations in derive-deftly; see
https://gitlab.torproject.org/Diziet/rust-derive-deftly/-/merge_requests/525
where I'm trying to start to address them.
|
| |
|
|
| |
We're going to reuse this.
|
| |
|
|
|
| |
As per
https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/3378#note_3277957
|
| |
|
|
| |
This makes it match the protocol and the spec.
|
| | |
|
| | |
|
| | |
|
| | |
|
| |
|
|
|
| |
We should use the same name everywhere, even if the name in the spec
is clumsy (as it is here).
|
| |
|
|
|
|
| |
We're going to want to apply the parse2 derive to Preamble, but this
field is anomalous. Rather than trying to make it regular, simply
move it to the old-parser-specific Consensus struct.
|
| |
|
|
|
|
|
| |
Consensus is not going to be easy to square with parse2 - especially,
signature handling is very different between the two parsers.
Instead, we'll have a new NetworkStatus type.
|
| |
|
|
|
| |
There is no such thing as the "header" according to the spec. We mean
the preamble.
|