Skip to content
This repository was archived by the owner on Sep 12, 2024. It is now read-only.

Commit 7006d9f

Browse files
nagashiyaahc
andauthored
Add meetings/2020-10-26.md (#19)
* Add meetings/2020-10-26.md * Update 2020-10-26.md Co-authored-by: Jane Lusby <jlusby42@gmail.com>
1 parent b4d9860 commit 7006d9f

File tree

1 file changed

+123
-0
lines changed

1 file changed

+123
-0
lines changed

meetings/2020-10-26.md

Lines changed: 123 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,123 @@
1+
# PEH 2020-10-26 Meeting Agenda
2+
3+
###### tags: `Error Handling` `Minutes`
4+
5+
- [2020-10-12 Meeting Minutes](https://hackmd.io/@rust-libs/HJ0hiQfDP)
6+
7+
## Attendees
8+
9+
- Jane Lusby
10+
- Ashley Mannix
11+
- Oliver :)
12+
- Jeremiah Senkpiel
13+
- Charles Ellis O'Riley Jr.
14+
- Sean Chen
15+
- Mara
16+
- DPC
17+
18+
## Action Items
19+
20+
- Implement trait based proof of concept for `Backtrace` in `core` (Jane Lusby)
21+
- Update `object-provider` API in generic member access RFC (Jane Lusby)
22+
- Create a tracking issue to document why `Box<dyn Error>` and related types do not implement the Error trait, what prevents it, and what would be needed moving forward (Ashley Mannix) [zulip topic](https://rust-lang.zulipchat.com/#narrow/stream/257204-project-error-handling/topic/impl.20Error.20or.20dyn.20Error.3F/near/213235983)
23+
- Fill out table of contents of expected sections in `The Rust Error Book` (Oliver)
24+
- Review long-term vision blog post (Jane Lusby, Jakub Duchniewicz, Ashley Mannix)
25+
- Begin process of stabilization of `Backtrace` and `fn backtrace` (Ashley Mannix)
26+
- Resolve last blockers in `Fix The Error Trait` tracking issue (Oliver)
27+
- Status update on [PanicInfo::message](https://github.com/rust-lang/rust/issues/66745) (DPC)
28+
- Get mentorship instructions for implementing [`Nicer Assert Messages`](https://github.com/rust-lang/rust/issues/44838) RFC and begin implementation (Charles Ellis O'Riley Jr. and DPC)
29+
- Add unstable `fn frames(&self) -> &[BacktraceFrame]` method to `Backtrace` on nightly based on `backtrace` crate's API (Sean Chen and Ashley Mannix)
30+
31+
## Agenda Items
32+
33+
- Review action items from last meeting
34+
- Begin tracking https://github.com/fusion-engineering-forks/rfcs/blob/panic/text/0000-panic-plan.md, status update from [@m-ou-se](https://github.com/m-ou-se)
35+
- Review open issues on project group repository
36+
- Anything to bubble up to the Libs meeting?
37+
38+
## Summary
39+
40+
### `impl<'a, E: Error + Sized + 'a> Error for &'a E`
41+
42+
- We believe we should pursue stabilizing this. There's already an open PR to update: https://github.com/rust-lang/rust/pull/75180
43+
44+
### `dyn Error` or `impl Error`?
45+
46+
- We think the recommendation of `&(dyn Error + 'static)` for borrowed abstract errors and `impl Error + Send + Sync + 'static` for owned abstract errors is reasonable.
47+
48+
### Table of contents for the Rust book
49+
50+
- We've sketched out a table of contents that's ready to PR back to the main repository.
51+
52+
### Stabilization for `Backtrace`
53+
54+
- There's a current question about whether it's a problem that `Backtrace` doesn't share much with the internal machinery used to print panic backtraces.
55+
- @KodrAus is looking into it, and will post an update on the stabilization PR.
56+
- The panic implementation is a little different because it doesn't require all frames to be captured upfront, which is important for panic-on-oom support. Even if we don't share everything we should be able to share a little more between `sys_common::backtrace` and `backtrace`.
57+
58+
### Fixing the `Error`
59+
60+
- The last outstanding item is that you can't use the precision flag to limit the number of frames being printed when formatting a backtrace. This doesn't need to block stabilization, and `Backtrace::frames` should also make it less important.
61+
62+
### `PanicInfo::message`
63+
64+
- It's currently in progress.
65+
66+
### Better assert RFC
67+
68+
- It's currently in progress.
69+
70+
### `Backtrace::frames`
71+
72+
- There's an open PR for it that we're working through.
73+
74+
### Vision blog post
75+
76+
- It's been updated with the latest round of feedback and is ready to go!
77+
78+
# Meeting Minutes
79+
80+
Meeting called to order at 13:00
81+
Review of Action Items
82+
Review of implementing the backtrace in core PoC and updating the object provider RFC. No updates from Jane Lusby on those two items. Jane made a separate trackingh issue for box dyn error.
83+
84+
Ashley Mannix will write a stabilization report for &dyn Error.
85+
86+
action item: adding a table of contents for the rust book to which oliver will add a PR to project-error-handling repo for adding a table of contents for the rust book.
87+
88+
@Ashley Mannix will investigate @ralfjung concerns abount pacin! relying on different internals than backtrace itself does
89+
90+
next action item: resolving the "Fix The Error Trait" issue by @oliver which he's still looking into. Subsequent conversation on this issue determined that "fix backtrace's fmt" has been done and that the fancy extra fmt features could be spli into seperate tracking issues.
91+
92+
next action item: @DPC status update on the "PanicInfo::message" PR which he started started working on it.
93+
94+
next action item: Charles Ellis O'Riley Jr. and @DPC have been working on implementing the Better assert RFC. @Charles Ellis O'Riley Jr forked and cloned the repo Copied and filled in the config.toml file and ran the x.py command which produced more errors that I need to work through.
95+
96+
next action item: backtrace frame. @Sean Chen started a draft PR for it.
97+
98+
agenda item: @Mara has opened a new RFC for fixing inconsistencies between core and std panic. In summary the proposal is to make std's and core's panic macro the same one, and make it behave identically to print!() in how it uses its arguments so panic!("{}") and panic!(123) etc. will no longer compile.
99+
100+
next agenda item: there are 7 open issues on the project group repo of which 3 have been discussed in this meeting, the impl Error for E: Error one and the backtrace proof of concept, and backtrace frames API
101+
102+
@Jakub Duchniewicz will check into this globally consistent error reporting
103+
104+
@Jane Lusby will look into Posix errors (the author)
105+
106+
* move std::io::Error to core is currently blocked
107+
* error return traces are blocked by the generic member access RFC
108+
109+
End of meeting.
110+
111+
112+
## Participants
113+
114+
People in attendance:
115+
Jane Lusby
116+
Ashley Mannix
117+
Oliver :cat2:
118+
Jeremiah Senkpiel
119+
DPC
120+
Mara
121+
Jakub Duchniewicz
122+
Sean Chen
123+
Charles Ellis O'Riley Jr.

0 commit comments

Comments
 (0)