microsoft / microsoft/sql-server-samples
pls fix check linux to use (CHAR*) cast for calls to CHECK_ERROR
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- PowerShell
- Sterne
- 11.2k
- Forks
- 9.1k
- Ø Merge
- 2 T. 7 Std.
- Gemergte PRs (30 T.)
- 14
Beschreibung
99% of people using C/Linux are running GCC.
GCC for a long time thows warnings if you do implicit conversion of string to char*. The preferred method is of course using const char*. There are two options.
1. change the check error strings to be typcasted as shown below
Current Version
CHECK_ERROR(retcode, "SQLAllocHandle(SQL_HANDLE_ENV)",
henv, SQL_HANDLE_ENV);
Type Casted Version
CHECK_ERROR(retcode,(CHAR*)"SQLAllocHandle(SQL_HANDLE_ENV)",
henv, SQL_HANDLE_ENV);
2. Add a statement between where const char gets defined as the string that is going to be passed to check error*
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Keine Datei und kein Test sind benannt. Beginne damit, die Aufrufstellen von CHECK_ERROR in den C/Linux-Beispielen zu finden, und untersuche die SQLAllocHandle-Beispiele auf die hier beschriebenen impliziten string-to-CHAR*-Warnungen. Als erledigt gilt die Aufgabe, wenn die betroffenen Aufrufe durchgängig den ausgewählten Ansatz verwenden und GCC diese Konvertierungen nicht mehr meldet.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- c, linux, sql
- Bereich
- databases, operating-systems
- Issue-Typ
- Bug
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100