| Commit message (Collapse) | Author | Age | Files | Lines | ||
|---|---|---|---|---|---|---|
| ... | ||||||
| * | | | netdoc parse2 tests: Avoid <String as TryFrom<Vec<u8>>> | Ian Jackson | 2025-09-29 | 1 | -1/+1 | |
| | | | | | | | | | | | | | Should fix the minimal versions test? | |||||
| * | | | netdoc parse2: Remove some unneeded allows | Ian Jackson | 2025-09-29 | 1 | -2/+0 | |
| | | | | ||||||
| * | | | netdoc parse2: Fix two clippy lints | Ian Jackson | 2025-09-29 | 2 | -1/+1 | |
| | | | | ||||||
| * | | | neteoc parse2: Provide Ignored value | Ian Jackson | 2025-09-29 | 3 | -1/+41 | |
| | | | | | | | | | | | | | | | | This will be used for some bits of netdocs where dummy values now appear. | |||||
| * | | | netdoc parse2: NetdocParseableFields: suppress errors for empty types | Ian Jackson | 2025-09-29 | 1 | -2/+10 | |
| | | | | ||||||
| * | | | neteoc parse2: Import earlier | Ian Jackson | 2025-09-29 | 1 | -2/+3 | |
| | | | | | | | | | | | | | | | | This is needed if we want to use these macros in `doc`, which we're going to. | |||||
| * | | | netdoc parse2: support `with` for rest arguemnts, and test | Ian Jackson | 2025-09-29 | 2 | -2/+18 | |
| | | | | ||||||
| * | | | netdoc parse2: support `with` for normal arguemnts, and test | Ian Jackson | 2025-09-29 | 2 | -7/+11 | |
| | | | | ||||||
| * | | | netdoc parse2: ArgumentSetMethods: take a parser function | Ian Jackson | 2025-09-29 | 2 | -13/+65 | |
| | | | | | | | | | | | | | We need this to support `with`. | |||||
| * | | | netdoc parse2: Support with for Item parsing, comprehensive tests | Ian Jackson | 2025-09-29 | 1 | -0/+64 | |
| | | | | ||||||
| * | | | netdoc parse2: tests: NeedsWith type | Ian Jackson | 2025-09-29 | 1 | -3/+55 | |
| | | | | | | | | | | | | | This type will be used to test `with` attributes. | |||||
| * | | | netdoc parse2: Support with for Item parsing | Ian Jackson | 2025-09-29 | 1 | -2/+23 | |
| | | | | | | | | | | | | | Tests will come in a moment. | |||||
| * | | | netdoc parse2: New single_item attribute, tests | Ian Jackson | 2025-09-29 | 1 | -6/+55 | |
| | | | | ||||||
| * | | | netdoc parse2: New single_item attribute | Ian Jackson | 2025-09-29 | 2 | -2/+30 | |
| | | | | | | | | | | | | | | | | | | | Many of the existing netdoc structs have items that are just the single argument. We don't want to turn them all into singleton tuples. Provide a macro feature for this instead. | |||||
| * | | | netdoc parse2: Break out ITEM_VALUE_FROM_UNPARSED | Ian Jackson | 2025-09-29 | 1 | -2/+6 | |
| | | | | | | | | | | | | | This is going to get more complicated. | |||||
| * | | | netdoc parse2: NetdocParseableFields: Fix default, and test | Ian Jackson | 2025-09-29 | 2 | -2/+9 | |
| | | | | ||||||
| * | | | netdoc parse2: NetdocParseableFields: introduce use of $fpatname | Ian Jackson | 2025-09-29 | 1 | -5/+4 | |
| | | | | | | | | | | | | | We're going to want to do more complex things here. | |||||
| * | | | netdoc parse2: Run selector-based trait check for intro item too | Ian Jackson | 2025-09-29 | 1 | -2/+6 | |
| | | | | ||||||
| * | | | netdoc parse2: Tests for new Object handling | Ian Jackson | 2025-09-29 | 1 | -1/+85 | |
| | | | | ||||||
| * | | | netdoc parse2: Fix Object handling (fmt) | Ian Jackson | 2025-09-29 | 1 | -2/+4 | |
| | | | | ||||||
| * | | | netdoc parse2: Fix Object handling | Ian Jackson | 2025-09-29 | 7 | -42/+133 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use the standard multiplicity technique rather than the ad-hoc impl on Option. This allows us to support an ad-hoc parsing function for a field that's `Option`. Disentangle the `label` field attribute, which did both setting the label, and expecting a different parsing approach: replace it with `with`. | |||||
| * | | | netdoc parse2: Fix label for PublicKey | Ian Jackson | 2025-09-29 | 1 | -1/+1 | |
| | | | | | | | | | | | | | | | | This mistake escaped because this wasn't actually checked by the code, but it's going to be checked in a moment. | |||||
| * | | | netdoc parse2: Fix "rest" item arguments, and test | Ian Jackson | 2025-09-29 | 3 | -5/+28 | |
| | | | | | | | | | | | | | | | | The previous code didn't compile at all, and this went unnoticed because it wasn't used by the poc. | |||||
| * | | | netdoc parse2: Actually use ArgumentSetSelector, and test | Ian Jackson | 2025-09-29 | 3 | -6/+12 | |
| | | | | | | | | | | | | | Without this, parsing of Option and Vec argument fields doesn't work. | |||||
| * | | | netdoc parse2: tests: Initial tests of ItemValueParseable macro | Ian Jackson | 2025-09-29 | 1 | -1/+76 | |
| | | | | | | | | | | | | | | | | This macro has some problems which I'm going to fix. Add a test case for the parts that are currently working. | |||||
| * | | | netdoc parse2: Fix typo in todo comment | Ian Jackson | 2025-09-29 | 1 | -1/+1 | |
| | | | | ||||||
| * | | | netdoc parse2: Don't duplicate the error source | Ian Jackson | 2025-09-29 | 1 | -1/+1 | |
| | | | | | | | | | | | | | `problem` is the `#[source]`. | |||||
| * | | | netdoc parse2: Fix invalid keyword error problem report | Ian Jackson | 2025-09-29 | 1 | -1/+1 | |
| | | | | | | | | | | | | | `#[from]` is not `#[source]`. We need to print the inner error. | |||||
| * | | | netdoc parse2: tests: Better error messages from error tests (fmt) | Ian Jackson | 2025-09-29 | 1 | -2/+10 | |
| | | | | | | | | | | | | | Worsify formatting as demanded by rustdoc. | |||||
| * | | | netdoc parse2: tests: Better error messages from error tests | Ian Jackson | 2025-09-29 | 1 | -2/+2 | |
| | | | | ||||||
| * | | | netdoc parse2: tests: Use ==== for dividers in output | Ian Jackson | 2025-09-29 | 1 | -1/+1 | |
| | | | | | | | | | | | | | Otherwise it's easy to get confuseed by ---- dividers for Objects. | |||||
| * | | | netdoc parse2: poc: Make test functions throw anyhow::Result | Ian Jackson | 2025-09-29 | 1 | -5/+4 | |
| | | | | | | | | | | | | | For reasons, TestResult doesn't print error sources. | |||||
| * | | | netdoc parse2: Fix keyword parsing to hate nul, not digit zero (!) | Ian Jackson | 2025-09-29 | 1 | -1/+1 | |
| | | | | ||||||
| * | | | netdoc parse2: Use $P::Result throughout | Ian Jackson | 2025-09-29 | 2 | -5/+6 | |
| | | | | | | | | | | | | | | | | This is needed when using these macros in a namespace with a local redefinition of `Result`. | |||||
| * | | | netdoc parse2: Fix a compile_error! invocation | Ian Jackson | 2025-09-29 | 1 | -1/+1 | |
| | | | | | | | | | | | | | I triggered this error and it didn't work right. | |||||
| * | | | remove default feature off sysinfo | trinity-1686a | 2025-09-28 | 1 | -1/+1 | |
| | | | | ||||||
| * | | | proto: Replace magic value with constant (fmt). | Gabriela Moldovan | 2025-09-25 | 1 | -1/+2 | |
| | | | | ||||||
| * | | | proto: Replace magic value with constant. | Gabriela Moldovan | 2025-09-25 | 1 | -4/+9 | |
| | | | | | | | | | | | | | | | | Also derives `CIRC_ACTION_COUNT` from the two other constants instead of hard-coding the value. | |||||
| * | | | proto: Use MAX_CONFLUX_LEGS instead of magic number. | Gabriela Moldovan | 2025-09-25 | 1 | -1/+2 | |
| | | | | | | | | | | | | | | | | Besides, it's better if we use the same number for the expected number of legs as we do in the conflux set impl. | |||||
| * | | | proto: Make padding and timeout actions take priority. | Gabriela Moldovan | 2025-09-25 | 1 | -34/+69 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This rewrites `next_circ_action()` yet again, using two layers of `PollAll`: * the inner layer drives an individual circuit leg. Each circuit has a `PollAll` that drives its futures * the outer layer drives the inner `PollAll`s belonging to the circuits that form the tunnel | |||||
| * | | | proto: Add comment emphasizing that the PollAll ordering matters. | Gabriela Moldovan | 2025-09-25 | 1 | -0/+7 | |
| | | | | ||||||
| * | | | proto: Add comment explaining why we need the chan_sender readiness check. | Gabriela Moldovan | 2025-09-25 | 1 | -0/+3 | |
| | | | | ||||||
| * | | | proto: Document exactly how PollAll polls its futures. | Gabriela Moldovan | 2025-09-25 | 1 | -1/+11 | |
| | | | | ||||||
| * | | | proto: Resolve a couple of clippy warnings in the tests. | Gabriela Moldovan | 2025-09-25 | 1 | -2/+2 | |
| | | | | ||||||
| * | | | proto: Rewrite should_skip_join_point to return bool. | Gabriela Moldovan | 2025-09-25 | 1 | -11/+10 | |
| | | | | | | | | | | | | | | | | We don't really need to return a `HopNum` anymore (because we work out the join point `HopNum` unconditionally in `next_circ_action`). | |||||
| * | | | proto: Avoid polling join point streams more than once. | Gabriela Moldovan | 2025-09-25 | 1 | -6/+11 | |
| | | | | | | | | | | | | | | | | | | | If we poll the ready streams on the join point more than once per reactor loop, we risk sending more than one DATA cell (which is not good, because cc might block after the first cell is sent). | |||||
| * | | | proto: Remove now-unused function. | Gabriela Moldovan | 2025-09-25 | 1 | -21/+2 | |
| | | | | ||||||
| * | | | proto: Rewrite ConfluxSet::next_circ_action() using PollAll. | Gabriela Moldovan | 2025-09-25 | 4 | -132/+109 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This removes our usage of `FuturesUnordered` in `ConfluxSet::next_circ_action()` to address two issues: * a fairness issue, where the futures driven by `FuturesUnordered` could be starved under some circumstances (#2180) * a logic error, where we'd explicitly avoid reading from the input channel if the outgoing `chan_sender` channel was blocked (#2179) Note that the fixing the latter will cause the reactor to buffer more into the unbounded `chan_sender` sink, but that *should* be okay, because no input message should be able cause us to queue cells excessively. Closes #2179, #2180 | |||||
| * | | | proto: Expose CircHopList in ConfluxSet. | Gabriela Moldovan | 2025-09-25 | 1 | -1/+1 | |
| | | | | | | | | | | | | | | | | We will soon need to access this directly (rather than via a method on `Circuit`) to work around borrow checker limitations. | |||||
| * | | | proto: Return multiple actions from next_circ_action (fmt). | Gabriela Moldovan | 2025-09-25 | 1 | -47/+47 | |
| | | | | ||||||
