google/benchmark

ICC sysinfo fixes

Open

#754 ouverte le 16 janv. 2019

Voir sur GitHub
 (1 commentaire) (0 réactions) (0 assignés)C++ (1 539 forks)batch import
good first issuehelp wanted

Métriques du dépôt

Stars
 (7 968 stars)
Métriques de merge PR
 (Merge moyen 4j 2h) (19 PRs mergées en 30 j)

Description

sysinfo fails to compile on OSX with ICC due to two issues:

error #2259 is caused by the return value from sysinfo.cc:558 being implicitly cast from unsigned long long to double.

error #177 is caused by methods at sysinfo.cc:112 ValueUnion::GetAsString and sysinfo:114 ValueUnion::GetAsInteger being unused.

To resolve this we should:

  1. add an explicit cast for hz to double
  2. add -wd177 to the ICC compiler flags

Guide contributeur

Direction de recherche
Ajoutez un cast explicite de hz en double à la ligne 558 de sysinfo.cc et ajoutez wd177 aux drapeaux du compilateur ICC.
Stack technique
cpp
Domaine
tooling
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
1
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C++Git
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
90