Skip to content

Commit 84fb06b

Browse files
authored
Add the 2025-06-04 Coding Guidelines Subcommittee Minutes (#341)
1 parent 85969af commit 84fb06b

1 file changed

Lines changed: 63 additions & 21 deletions

File tree

  • subcommittee/coding-guidelines/meetings/2025-06-04

subcommittee/coding-guidelines/meetings/2025-06-04/minutes.md

Lines changed: 63 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -10,50 +10,92 @@
1010

1111
## Agenda
1212

13-
0. Solicitation of notetaker
13+
0. Solicitation of notetaker
1414
1. Acceptance of [Previous Meeting Minutes](https://github.com/rustfoundation/safety-critical-rust-consortium/blob/main/subcommittee/coding-guidelines/meetings/2025-05-28/minutes.md)
15-
2. Introduction of new members
16-
3. Review of [revised milestone](https://github.com/rustfoundation/safety-critical-rust-coding-guidelines/milestone/1) focused on broad + deep approach to macros chapter - 5 mins
17-
4. Discuss work Alex Celeste has on-going for rewriting style guideline - 20 mins
18-
5. Working session - 20 mins
19-
* Let's try to think of 1-2 guidelines ahead of time we'd like to jot down, macros are great, wider also fine
20-
* Note! We'd like to aim for unambiguous guidelines at this point to build confidence in their quality
21-
* Spin off PRs to rough them in - volunteers
22-
6. Review session:
23-
* Review open [PRs](https://github.com/rustfoundation/safety-critical-rust-coding-guidelines/pulls) & [issues](https://github.com/rustfoundation/safety-critical-rust-coding-guidelines/issues) - 5 mins
24-
7. Round table - 5 mins
15+
2. Introduction of new members
16+
3. Review of [revised milestone](https://github.com/rustfoundation/safety-critical-rust-coding-guidelines/milestone/1) focused on broad \+ deep approach to macros chapter \- 5 mins
17+
4. Discuss work Alex Celeste has on-going for rewriting style guideline \- 20 mins
18+
5. Working session \- 20 mins
19+
* Let's try to think of 1-2 guidelines ahead of time we'd like to jot down, macros are great, wider also fine
20+
* Note\! We'd like to aim for unambiguous guidelines at this point to build confidence in their quality
21+
* Spin off PRs to rough them in \- volunteers
22+
6. Review session:
23+
* Review open [PRs](https://github.com/rustfoundation/safety-critical-rust-coding-guidelines/pulls) & [issues](https://github.com/rustfoundation/safety-critical-rust-coding-guidelines/issues) \- 5 mins
24+
7. Round table \- 5 mins
2525

2626
## Check-in area
2727

2828
**Please add your name, and an emoji that describes your day.**
2929

30-
*
30+
* Andrew Fernandes 🤖☕
31+
* Samuel Wright 😪
32+
* Enow Scott 🙂
33+
* Pete LeVasseur, 🤹
34+
* Douglas Deslauriers ⛵
35+
* Sasha Pourcelot 🚄
36+
* El Mahdi El Araby 🔥
37+
* David Svoboda 😟
38+
* Tiago Manczak
39+
* Markus Hosch
40+
* Arthur Hicken
41+
* Alex Celeste ☕
42+
* Lukas Wirth
3143

3244
**Notetaker:**
3345

34-
*
46+
* Andrew Fernandes
47+
48+
## Coding Guidelines Thoughts {#coding-guidelines-thoughts}
49+
50+
* Do not divide by 0 (CERT INT33-C)
51+
* \[todo\] by Douglas
52+
* Avoid lossy conversions (CERT INT31-C and FLP34-C)
53+
* \[todo\] by Alex Celeste
54+
* Do not allow Injections (SQL, XSS, OS Command injection, etc.)
55+
* \[todo\] by David
56+
* Do not dereference a raw pointer that is dangling or unaligned ([FLS](https://rust-lang.github.io/fls/expressions.html#fls_5cm4gkt55hjh))
57+
* \[todo\] by
58+
* Do not allow recursion (advisory?) \- there’s a clippy lint that was suggested
59+
* \[todo\] by Oreste
60+
* Static stack size analysis must be done
61+
* Note that `emit-stack-sizes` is [nightly-only](https://github.com/rust-lang/rust/issues/54192), not slated for stabilization
62+
* Recursion is currently allowed, tail-call elision is not guaranteed
63+
* Might *have* to be derived from static object-code analysis
64+
* Prefer macros which could be expanded and included as expanded source should safety-qualification needs dictate (advisory?)
65+
* \[todo\] by Sasha
3566

3667
## Housekeeping section
3768

3869
* Document space: [coding-guidelines](https://github.com/rustfoundation/safety-critical-rust-consortium/tree/main/subcommittee/coding-guidelines)
39-
* Zulip: [safety-critical-consortium: Coding Guidelines](https://rust-lang.zulipchat.com/#narrow/channel/445688-safety-critical-consortium/topic/Coding.20Guidelines)
40-
* [Early Alpha Milestone](https://github.com/rustfoundation/safety-critical-rust-coding-guidelines/milestone/1)
41-
* [Work Item Board](https://github.com/orgs/rustfoundation/projects/1)
70+
* Zulip: [safety-critical-consortium: Coding Guidelines](https://rust-lang.zulipchat.com/#narrow/channel/445688-safety-critical-consortium/topic/Coding.20Guidelines)
71+
* [Early Alpha Milestone](https://github.com/rustfoundation/safety-critical-rust-coding-guidelines/milestone/1)
4272

4373
## Tasks
4474

45-
* See below for the [todo] markers
75+
* See **above** for the \[todo\] markers
4676

4777
## Meeting Minutes
4878

49-
*
79+
* Last week’s meeting minutes were accepted
80+
* Intro for the (absent) new members
81+
* Review of the [revised milestone](https://github.com/rustfoundation/safety-critical-rust-coding-guidelines/milestone/1) board, emphasis that dates are advisory only
82+
* Discussion on the ongoing style guideline rewrite
83+
* Discussion of what goes into the guideline
84+
* *Example* blanket requirement as per MISRA style
85+
* Authors are asked to think about *who* the rule is going to target
86+
* Appeal that more people will be *reading* the guideline rather than writing it
87+
* Mandatory and Required guidelines are the priority
88+
* Ask to read [PR \#27](https://github.com/rustfoundation/safety-critical-rust-coding-guidelines/pull/127) and comment for everyone
89+
* Working session discussion
90+
* See [Coding Guidelines Thoughts](#coding-guidelines-thoughts), above
91+
* Discussion re procedural vs declarative macros: hygiene is much more complex and nuanced than may be first realized
92+
* Macro hygiene is fundamentally a tooling problem
93+
* Ask for volunteers to *test-run* the issue template on GitHub
5094

5195
## Material
5296

5397
Any material to read before the meeting should be included here.
5498

5599
### GitHub Project Board for Work Items
56100

57-
* [Work Item Board](https://github.com/orgs/rustfoundation/projects/1)
58-
* [Coding guidelines published](https://github.com/rustfoundation/safety-critical-rust-consortium/issues/188#issue-2869798433): Proposes a way of work to tackle the coding guidelines
59-
101+
* [Work Item Board](https://github.com/orgs/rustfoundation/projects/1)

0 commit comments

Comments
 (0)