owenca wrote:
Should `Leave` apply to the hash only or also to the name after the hash?
https://github.com/llvm/llvm-project/pull/139750
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/gedare updated
https://github.com/llvm/llvm-project/pull/139750
>From 5974807cae2cf08b26cadb2fce3bd0236519f3c0 Mon Sep 17 00:00:00 2001
From: Gedare Bloom
Date: Tue, 13 May 2025 08:42:41 -0600
Subject: [PATCH 1/5] [clang-format] Add IndentPPDirectives Leave option
Allow an o
https://github.com/mydeveloperday approved this pull request.
I approve if @owenca does.
https://github.com/llvm/llvm-project/pull/139750
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commi
@@ -1142,7 +1144,8 @@ void UnwrappedLineParser::parsePPEndIf() {
// If the #endif of a potential include guard is the last thing in the file,
// then we found an include guard.
if (IncludeGuard == IG_Defined && PPBranchLevel == -1 && Tokens->isEOF() &&
- Style.Indent
https://github.com/gedare updated
https://github.com/llvm/llvm-project/pull/139750
Rate limit · GitHub
body {
background-color: #f6f8fa;
color: #24292e;
font-family: -apple-system,BlinkMacSystemFont,Segoe
UI,Helvetica,Arial,sans-se
gedare wrote:
This change can help support some of the goals stated in #35368 but doesn't
exactly capture the intent of that Issue, which adds a requirement to make the
PP indent level also advance the indent level. I am not currently interested in
that particular feature.
https://github.com/
https://github.com/gedare edited
https://github.com/llvm/llvm-project/pull/139750
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/gedare edited
https://github.com/llvm/llvm-project/pull/139750
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/gedare created
https://github.com/llvm/llvm-project/pull/139750
Allow an option to leave preprocessor directive indenting as-is. This
simplifies handling mixed styles of CPP directive indentation.
Fixes #38511
Rate limit · GitHub
bod
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Gedare Bloom (gedare)
Changes
Allow an option to leave preprocessor directive indenting as-is. This
simplifies handling mixed styles of CPP directive indentation.
Fixes #38511
---
Full diff: https://github.com/llvm/llvm-project/pull/139
10 matches
Mail list logo
