pls fix check linux to use (CHAR*) cast for calls to CHECK_ERROR

Aperta
#500 1 commento 1 reazione 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
3/5
Tempo stimato
1-2 giorni
Idoneità per principianti
35/100
Tipo di issue
Bug
Chiarezza
Abbastanza chiara
Stato di attività
Ferma
Stack tecnologico
c, linux, sql
Ambito
databases, operating-systems

Direzione di ricerca

Non sono specificati né file né test. Inizia individuando i punti di chiamata di CHECK_ERROR negli esempi C/Linux e esamina gli esempi SQLAllocHandle alla ricerca degli avvisi impliciti string-to-CHAR* descritti qui. Il lavoro è completato quando le chiamate interessate usano in modo coerente l’approccio selezionato e GCC non segnala più queste conversioni.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

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*

Lingua principale
PowerShell
Stelle
11.2k
Fork
9.1k
Merge medio
2g 7h
PR unite (30g)
14

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di microsoft/sql-server-samples

Tutte le issue di microsoft/sql-server-samples

Issue simili

Altre issue su Databases

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.