public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Rebecca Cran" <rebecca@bsdio.com>
To: devel@edk2.groups.io, Michael Kinney <michael.d.kinney@intel.com>,
	Sean Brogan <spbrogan@outlook.com>,
	Oliver Smith-Denny <osde@linux.microsoft.com>,
	Pedro Falcato <pedro.falcato@gmail.com>
Cc: Rebecca Cran <rebecca@bsdio.com>
Subject: [edk2-devel] [PATCH edk2-CCodingStandardsSpecification v3 0/1] Prefer use of `static` C keyword over EDK2 type `STATIC`
Date: Mon,  3 Feb 2025 11:45:47 -0700	[thread overview]
Message-ID: <20250203184548.2528198-1-rebecca@bsdio.com> (raw)

Prefer use of `static` C keyword over EDK2 type `STATIC`

A while ago a decision was made on the edk2-devel mailing list that
the `STATIC` EDK2 type should be replaced with the C keyword `static`.

Following further discussion, it was decided to disallow static local
variables to prevent breakage when using GoogleTest.

Update the Coding Specification to match. While here, remove the
outdated section disallowing static functions since they're no longer
a problem.

Changes from v2 to v3:

Dropped the update to book.json.
Added contributed-under line to commit message.

Rebecca Cran (1):
  Prefer use of `static` C keyword over EDK2 type `STATIC`

 5_source_files/54_code_file_structure.md    | 9 ++-------
 5_source_files/56_declarations_and_types.md | 2 +-
 README.md                                   | 3 ++-
 3 files changed, 5 insertions(+), 9 deletions(-)

-- 
2.48.1



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#121075): https://edk2.groups.io/g/devel/message/121075
Mute This Topic: https://groups.io/mt/110977228/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-



             reply	other threads:[~2025-02-03 18:46 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-02-03 18:45 Rebecca Cran [this message]
2025-02-03 18:45 ` [edk2-devel] [PATCH edk2-CCodingStandardsSpecification v3 1/1] Prefer use of `static` C keyword over EDK2 type `STATIC` Rebecca Cran
2025-02-03 19:09   ` Michael D Kinney via groups.io

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-list from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20250203184548.2528198-1-rebecca@bsdio.com \
    --to=devel@edk2.groups.io \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox