[clang-tools-extra] [clang-doc][NFC] refactor out file helpers (PR #134298)

2025-04-07 Thread Paul Kirth via cfe-commits
@@ -0,0 +1,72 @@ +//===-- FileHelpersClangDoc.cpp - File Helpers ---*- C++-*-===// ilovepi wrote: Apparently, we don't use that line anymore. https://llvm.org/docs/CodingStandards.html#file-headers Was brought up here when I saw this the first

[clang-tools-extra] [clang-doc][NFC] refactor out file helpers (PR #134298)

2025-04-07 Thread Paul Kirth via cfe-commits
https://github.com/ilovepi edited https://github.com/llvm/llvm-project/pull/134298 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [clang-doc][NFC] refactor out file helpers (PR #134298)

2025-04-07 Thread Daniel Thornburgh via cfe-commits
@@ -0,0 +1,72 @@ +//===-- FileHelpersClangDoc.cpp - File Helpers ---*- C++-*-===// mysterymath wrote: The filename in this comment should match the actual filename. https://github.com/llvm/llvm-project/pull/134298 ___

[clang-tools-extra] [clang-doc][NFC] refactor out file helpers (PR #134298)

2025-04-07 Thread Paul Kirth via cfe-commits
https://github.com/ilovepi edited https://github.com/llvm/llvm-project/pull/134298 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [clang-doc][NFC] refactor out file helpers (PR #134298)

2025-04-05 Thread via cfe-commits
https://github.com/PeterChou1 updated https://github.com/llvm/llvm-project/pull/134298 >From fa35468f673ace035036a1c15d2d6ab756c2581e Mon Sep 17 00:00:00 2001 From: PeterChou1 Date: Thu, 3 Apr 2025 15:59:34 -0400 Subject: [PATCH 1/3] factor out file helpers --- clang-tools-extra/clang-doc/CMa

[clang-tools-extra] [clang-doc][NFC] refactor out file helpers (PR #134298)

2025-04-04 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang-tools-extra Author: None (PeterChou1) Changes Split from https://github.com/llvm/llvm-project/pull/133161 refactor the code to extract file helpers used in HTML generators for use in other generators for clang-doc --- Full diff: https://github.c

[clang-tools-extra] [clang-doc][NFC] refactor out file helpers (PR #134298)

2025-04-03 Thread Paul Kirth via cfe-commits
https://github.com/ilovepi edited https://github.com/llvm/llvm-project/pull/134298 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [clang-doc][NFC] refactor out file helpers (PR #134298)

2025-04-03 Thread Paul Kirth via cfe-commits
@@ -0,0 +1,25 @@ +//===-- File.h --- File Helpers ---*- C++-*-===// +// +// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions. +// See https://llvm.org/LICENSE.txt for license information. +// SPDX-License-Identifier: Apache-2.0 WITH LLV

[clang-tools-extra] [clang-doc][NFC] refactor out file helpers (PR #134298)

2025-04-03 Thread Paul Kirth via cfe-commits
https://github.com/ilovepi approved this pull request. Since this just moves common code I think this is fine for now. We should revisit some of the interfaces though, especially the ones that return SmallString<128>. I think there is also a Path API that handles relative paths, so we can per

[clang-tools-extra] [clang-doc][NFC] refactor out file helpers (PR #134298)

2025-04-03 Thread Paul Kirth via cfe-commits
@@ -0,0 +1,25 @@ +//===-- File.h --- File Helpers ---*- C++-*-===// +// +// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions. +// See https://llvm.org/LICENSE.txt for license information. +// SPDX-License-Identifier: Apache-2.0 WITH LLV

[clang-tools-extra] [clang-doc][NFC] refactor out file helpers (PR #134298)

2025-04-03 Thread via cfe-commits
https://github.com/PeterChou1 updated https://github.com/llvm/llvm-project/pull/134298 >From fa35468f673ace035036a1c15d2d6ab756c2581e Mon Sep 17 00:00:00 2001 From: PeterChou1 Date: Thu, 3 Apr 2025 15:59:34 -0400 Subject: [PATCH 1/4] factor out file helpers --- clang-tools-extra/clang-doc/CMa

[clang-tools-extra] [clang-doc][NFC] refactor out file helpers (PR #134298)

2025-04-03 Thread via cfe-commits
https://github.com/PeterChou1 updated https://github.com/llvm/llvm-project/pull/134298 >From fa35468f673ace035036a1c15d2d6ab756c2581e Mon Sep 17 00:00:00 2001 From: PeterChou1 Date: Thu, 3 Apr 2025 15:59:34 -0400 Subject: [PATCH 1/5] factor out file helpers --- clang-tools-extra/clang-doc/CMa

[clang-tools-extra] [clang-doc][NFC] refactor out file helpers (PR #134298)

2025-04-03 Thread via cfe-commits
github-actions[bot] wrote: :warning: C/C++ code formatter, clang-format found issues in your code. :warning: You can test this locally with the following command: ``bash git-clang-format --diff HEAD~1 HEAD --extensions cpp,h -- clang-tools-extra/clang-doc/support/File.cpp clang

[clang-tools-extra] [clang-doc][NFC] refactor out file helpers (PR #134298)

2025-04-03 Thread via cfe-commits
https://github.com/PeterChou1 updated https://github.com/llvm/llvm-project/pull/134298 >From fa35468f673ace035036a1c15d2d6ab756c2581e Mon Sep 17 00:00:00 2001 From: PeterChou1 Date: Thu, 3 Apr 2025 15:59:34 -0400 Subject: [PATCH 1/2] factor out file helpers --- clang-tools-extra/clang-doc/CMa

[clang-tools-extra] [clang-doc][NFC] refactor out file helpers (PR #134298)

2025-04-03 Thread via cfe-commits
https://github.com/PeterChou1 created https://github.com/llvm/llvm-project/pull/134298 Split from https://github.com/llvm/llvm-project/pull/133161 refactor the code to extract file helpers used in HTML generators for use in other generators for clang-doc >From fa35468f673ace035036a1c15d2d6ab7