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

Ouverte
#500 1 commentaire 1 réaction 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Évaluation

Difficulté
3/5
Temps estimé
1-2 jours
Accessibilité débutants
35/100
Type d'issue
Bug
Clarté
Plutôt claire
Activité
À l'abandon
Stack technique
c, linux, sql
Domaine
databases, operating-systems

Piste de recherche

Aucun fichier ni test n’est nommé. Commencez par localiser les sites d’appel de CHECK_ERROR dans les exemples C/Linux et examinez les exemples SQLAllocHandle à la recherche des avertissements implicites string-to-CHAR* décrits ici. Le travail est considéré comme terminé lorsque les appels concernés utilisent systématiquement l’approche sélectionnée et que GCC ne signale plus ces conversions.

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

Description

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*

Langage dominant
PowerShell
Étoiles
11.2k
Forks
9.1k
Merge moyen
2 j 7 h
PR mergées (30 j)
14

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de microsoft/sql-server-samples

Toutes les issues de microsoft/sql-server-samples

Issues similaires

Plus d'issues Databases

Recevez les nouvelles issues par e-mail

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