|
| 1 | +# **Coding Guidelines Subcommittee Meeting on 2025-12-03 @ 0800 UTC / 0900 CET / 1700 JST / 0300 EST** |
| 2 | + |
| 3 | +[Link](https://www.worldtimebuddy.com/?qm=1&lid=14,12,1850147,5&h=14&date=2026-01-14&sln=8-9&hf=1) to meeting time in common time zones. |
| 4 | + |
| 5 | +| Search Key | Description | |
| 6 | +| :---- | :---- | |
| 7 | +| \[todo\] | Action Item | |
| 8 | +| \[decision\] | Something decided on | |
| 9 | +| \[important\] | Key information | |
| 10 | + |
| 11 | +## **Agenda** |
| 12 | + |
| 13 | +1. Solicitation of notetaker |
| 14 | +2. Acceptance of [Previous Meeting Minutes](https://github.com/rustfoundation/safety-critical-rust-consortium/blob/main/subcommittee/coding-guidelines/meetings/2025-12-03/minutes.md) |
| 15 | +3. Introduction of new members |
| 16 | +4. Review progress on guidelines incorporated from CERT |
| 17 | + * High-level mapping of CERT rule groups to Rust |
| 18 | + * Integers - Félix Fischer |
| 19 | + * Arrays - Alex Celeste |
| 20 | + * Floating Point - Andrew Fernandes |
| 21 | +5. Proposals and ideas for new rules |
| 22 | +6. Progress on ongoing tasks |
| 23 | + * PR https://github.com/rustfoundation/safety-critical-rust-coding-guidelines/pull/256 seems stalled. Should we assign someone? |
| 24 | + * PR https://github.com/rustfoundation/safety-critical-rust-coding-guidelines/pull/240 seems stalled. Should we assign someone? |
| 25 | + * Issue https://github.com/rustfoundation/safety-critical-rust-coding-guidelines/issues/29 should be split into coding guidelines. Discuss their headlines. |
| 26 | +7. Round table |
| 27 | + |
| 28 | +## **Check-in area** |
| 29 | + |
| 30 | +* Jonatan H. Zeidler ☺️ |
| 31 | +* Roberto Bagnara 🏃♂️ |
| 32 | +* Markus Hosch 🧐 |
| 33 | +* Fernando Jose 🙂 |
| 34 | +* William Barsse |
| 35 | + |
| 36 | +**Please add your name, and an emoji that describes your day.** |
| 37 | + |
| 38 | +**Notetaker:** |
| 39 | + |
| 40 | +* Jonatan H. Zeidler |
| 41 | + |
| 42 | +For tips on how we take notes in the Safety-Critical Rust Consortium, please see the [Meeting Notetaker Role](https://github.com/rustfoundation/safety-critical-rust-consortium/blob/main/docs/notetaker-role.md) doc. |
| 43 | + |
| 44 | +## **Housekeeping section** |
| 45 | + |
| 46 | +* Document space: [coding-guidelines](https://github.com/rustfoundation/safety-critical-rust-consortium/tree/main/subcommittee/coding-guidelines) |
| 47 | +* Zulip: [safety-critical-consortium: Coding Guidelines](https://rust-lang.zulipchat.com/#narrow/channel/445688-safety-critical-consortium/topic/Coding.20Guidelines) |
| 48 | +* [Kanban board](https://github.com/orgs/rustfoundation/projects/1/views/3) |
| 49 | + * [`contributor experience`](https://github.com/orgs/rustfoundation/projects/1/views/4) view |
| 50 | + * [`coding guideline`](https://github.com/orgs/rustfoundation/projects/1/views/5) view |
| 51 | + |
| 52 | +## **Tasks** |
| 53 | + |
| 54 | +Markus Hosch: Find out whether and where we have rules on how to structure a guideline (e.g. order of sections, length of rationale etc.). This should probably be in Contributing.md. |
| 55 | +Oreste Bernardi: Take ownership of PR 256 (foster the completion) |
| 56 | +Markus Hosch: talk with Pete about whether there is an issue in CI (PR 240 and 256 seem blocked) |
| 57 | +Markus Hosch: will turn Issue #29 into separate issues |
| 58 | + |
| 59 | +## **Meeting Minutes** |
| 60 | + |
| 61 | +1. Solicitation of notetaker -> done |
| 62 | +2. Acceptance of Previous Meeting Minutes -> accepted |
| 63 | +3. 2 new members introduced themselves (Jonatan H. Zeidler and Roberto Bagnara) |
| 64 | +4. Review progress on guidelines incorporated from CERT |
| 65 | + * Involved people are not present -> skipping |
| 66 | +5. Progress on ongoing tasks |
| 67 | + * PR https://github.com/rustfoundation/safety-critical-rust-coding-guidelines/pull/256 seems stalled. Should we assign someone? |
| 68 | + * Seems rather complete (also: correctness over completeness) |
| 69 | + * Contributing.md does not have anything about structure (e.g. about the need and extend of a rationale) |
| 70 | + * Oreste Bernardi takes ownership (as main reviewer) |
| 71 | + * PR https://github.com/rustfoundation/safety-critical-rust-coding-guidelines/pull/240 seems stalled. Should we assign someone? |
| 72 | + * Issue https://github.com/rustfoundation/safety-critical-rust-coding-guidelines/issues/29 should be split into coding guidelines. Discuss their headlines. |
| 73 | + * Discussed first bullet point about offsetting raw pointers. There was no clear consensus about whether raw pointers to zero sized types should be prohibited in the first place or just offset operations on it. |
| 74 | + * \*const/mut () might be simply incorrect when used for type erasure, because `()` is the unit type being a unique value not in memory |
| 75 | + * `core::ffi::c_void` is not zero-sized btw |
| 76 | + * Markus will split the issue into multiple issues |
| 77 | +6. Proposals and ideas for new rules |
| 78 | +7. Round table |
| 79 | + * A question was raised about central coordination of writing the guidelines -> there is a Kanban board in Github |
| 80 | + * A question was raised about the ultimate goal |
| 81 | + * limit what you can with the language via guideline and enable tool committee to enforce this by lints |
| 82 | + * On the one hand it is meant to prevent pitfalls, on the other hand being even stricter by disallowing correct code patterns to reduce complexity and thus improve safety, there may be strict rules as well as enforced user based decisions |
| 83 | + |
| 84 | + |
| 85 | +## **Material** |
| 86 | + |
| 87 | +Any material to read before the meeting should be included here. |
| 88 | + |
| 89 | +* Milestone: [Prepare for launch to wider Rust community](https://github.com/rustfoundation/safety-critical-rust-coding-guidelines/milestone/1) |
| 90 | +* FLS Maintenance: [FLS Team \- North Star](https://hackmd.io/@plevasseur/HJb6qomOge/edit) |
| 91 | +* [Github Issue to discuss about panicking in Safety Critical](https://github.com/rustfoundation/safety-critical-rust-coding-guidelines/issues/158) |
| 92 | + |
0 commit comments