https://bugs.kde.org/show_bug.cgi?id=387272
Daniel Santos <daniel.san...@pobox.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |daniel.san...@pobox.com
--- Comment #9 from Daniel Santos <daniel.san...@pobox.com> ---
Yeah, this is annoying as all hell. (I'm using KDevelop 5.4.5 with gcc 9.) For
example, when editing include/uapi/asm-generic/termb8its.h from Linux, the damn
pop up appears anywhere I move my mouse, thus preventing me from clicking on
what's behind it.
A note here, it is guarded with:
#ifndef __ASM_GENERIC_TERMBITS_H
#define __ASM_GENERIC_TERMBITS_H
...
#endif /* __ASM_GENERIC_TERMBITS_H */
and choosing Solution 2 adds:
#ifndef TERMBITS_H_INCLUDED
#define TERMBITS_H_INCLUDED
...
#endif // TERMBITS_H_INCLUDED
This is bad because this is a C file and // is not allowed in the kernel.
Also, the code should not assume that it is not guarded if the macro used does
BTW fit some specific format that is an arbitrary choice -- we should never
push our personal coding standards or conventions on others. (FYI, yes I know
that Linux is violating the C standard by using macros and other identifiers
that start with _ or __.)
In my case, I have set my C parser compiler to gcc and my command line args to
this custom value (derived from the kernel make):
-ferror-limit=100 -fspell-checking -Wdocumentation -Wunused-parameter
-Wunreachable-code -Wall -nostdinc -isystem
/home/daniel/proj/openwrt/staging_dir/toolchain-mipsel_24kc_gcc-7.3.0_musl/lib/gcc/mipsel-openwrt-linux-musl/7.3.0/include
-I/home/daniel/proj/kernel/4.14/arch/mips/include
-I/home/daniel/proj/kernel/4.14/arch/mips/include/generated
-I/home/daniel/proj/kernel/4.14/include
-I/home/daniel/proj/kernel/4.14/arch/mips/include/uapi
-I/home/daniel/proj/kernel/4.14/arch/mips/include/generated/uapi
-I/home/daniel/proj/kernel/4.14/include/uapi
-I/home/daniel/proj/kernel/4.14/include/generated/uapi -include
/home/daniel/proj/kernel/4.14/include/linux/kconfig.h -D__KERNEL__
-DVMLINUX_LOAD_ADDRESS=0xffffffff80000000 -DDATAOFFSET=0 -Wall -Wundef
-Wstrict-prototypes -Wno-trigraphs -m32 -fno-strict-aliasing -fno-common
-fshort-wchar -Werror-implicit-function-declaration -Wno-format-security
-std=gnu89 -fno-PIE -fno-pic -pipe -msoft-float -DGAS_HAS_SET_HARDFLOAT
-Wa,-msoft-float -ffreestanding -fno-stack-check -march=i386 -Wa,--trap
-DTOOLCHAIN_SUPPORTS_VIRT
-I/home/daniel/proj/kernel/4.14/arch/mips/include/asm/mach-ralink
-I/home/daniel/proj/kernel/4.14/arch/mips/include/asm/mach-ralink/mt7620
-I/home/daniel/proj/kernel/4.14/arch/mips/include/asm/mach-generic
-fno-asynchronous-unwind-tables -fno-delete-null-pointer-checks
-Wno-frame-address -Wno-format-truncation -Wno-format-overflow
-Wno-int-in-bool-context -Os -Wno-maybe-uninitialized
--param=allow-store-data-races=0 -DCC_HAVE_ASM_GOTO -Wframe-larger-than=1024
-fno-stack-protector -Wno-unused-but-set-variable -Wno-unused-const-variable
-fomit-frame-pointer -fno-var-tracking-assignments
-Wdeclaration-after-statement -Wno-pointer-sign -fno-strict-overflow
-fno-merge-all-constants -fmerge-constants -fno-stack-check -fconserve-stack
-Werror=implicit-int -Werror=strict-prototypes -Werror=date-time
-Werror=incompatible-pointer-types -Werror=designated-init -fverbose-asm -S
Thank you!
PS: Yes, I'm lying to gcc and having it parse the mipsel architecture, but only
for code formatting since there's not a way to specify the actual gcc compiler
used for parsing. So I'm telling it the arch is i386 to get it to use 32-bit
ints and pointers.
--
You are receiving this mail because:
You are watching all bug changes.
