4paradigm / 4paradigm/OpenMLDB

build: compile with more strict flags

Ouverte
#1,528 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
enhancement
Langage dominant
C++
Étoiles
1.7k
Forks
331
Merge moyen
12 j 12 h
PR mergées (30 j)
1

Description

**Describe the feature you'd like**

It appear there are many warnings during compilation. To improve code quality, we should consider just failing the compile when some of the warnings meet, not limited to e.g:
- -Wdangling-gsl
- -Wformat
- -Wdelete-non-abstract-non-virtual-dtor
- -Wthread-safety

**Additional context**

Add any other context or screenshots about the feature request here.

Guide de contribution

Ouvrir le guide de contribution

Piste de recherche

Look at the project's build configuration files (CMakeLists.txt, Makefile, or similar) to see current compiler flags. Research each suggested warning flag (-Wdangling-gsl, -Wformat, -Wdelete-non-abstract-non-virtual-dtor, -Wthread-safety) for compatibility with the codebase. Test compilation with these flags enabled to identify and understand the warnings, then decide which to promote to errors.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
cpp
Domaine
build-system
Type d'issue
Fonctionnalité
Difficulté
3/5
Temps estimé
1-2 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
45/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.