rniwa wrote:
Closing this in favor of https://github.com/llvm/llvm-project/pull/91103.
https://github.com/llvm/llvm-project/pull/90552
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/rniwa closed https://github.com/llvm/llvm-project/pull/90552
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
haoNoQ wrote:
Hmm it doesn't look like we actually have that other suppression. It looks like
the checker simply never warned in the standard library because there aren't
WebKit classes in the standard library. But it looks like you've found a few
cases where it actually happens? How does
rniwa wrote:
> Ok am I reading this right: this is about testing code that lives directly in
> namespace `std::` but somehow still lives in your project right? Such as your
> custom modifications to standard classes, various overloads and
> specializations of standard things.
No, this is
haoNoQ wrote:
(If yes, LGTM!)
https://github.com/llvm/llvm-project/pull/90552
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
haoNoQ wrote:
Ok am I reading this right: this is about testing code that lives directly in
namespace `std::` but somehow still lives in your project right? Such as your
custom modifications to standard classes, various overloads and specializations
of standard things.
This patch does not
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Ryosuke Niwa (rniwa)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/90552.diff
4 Files Affected:
- (modified)
clang/lib/StaticAnalyzer/Checkers/WebKit/UncountedCallArgsChecker.cpp (+6)
- (modified)
https://github.com/rniwa created https://github.com/llvm/llvm-project/pull/90552
None
>From fa7a6e376b07ae6262dd06920c87dc69705a646d Mon Sep 17 00:00:00 2001
From: Ryosuke Niwa
Date: Mon, 29 Apr 2024 20:24:24 -0700
Subject: [PATCH] Skip over std namespace in WebKit checkers.
---