rodrigodcastro
commented
5 years ago Gracias Pablo. Pregunta: lo probó alguen además de vos en entorno fuera de mac ? (Por ejemplo Freddy) Para saber. Saludos !
El 19/11/2018 13:40, "Pablo" notifications@github.com escribió:
Some imports are placed differently inside MacOS C standard library, like hashmap. Also, since the default C++ compiles is Clang, some warning are raised as errors. A fix is provided for them.
You can view, comment on, or merge this pull request online at:
https://github.com/SimulationEverywhere/CDPP_ExtendedStates-codename-Santi/pull/5 Commit Summary
- Hashmap include
- Missing compatibility fixes
File Changes
- M src/cd++/evt/event.h https://github.com/SimulationEverywhere/CDPP_ExtendedStates-codename-Santi/pull/5/files#diff-0 (2)
- M src/cd++/model/pmodeladm.h https://github.com/SimulationEverywhere/CDPP_ExtendedStates-codename-Santi/pull/5/files#diff-1 (9)
- M src/cd++/parser/tokit.h https://github.com/SimulationEverywhere/CDPP_ExtendedStates-codename-Santi/pull/5/files#diff-2 (3)
- M src/cd++/utils/prnutil.h https://github.com/SimulationEverywhere/CDPP_ExtendedStates-codename-Santi/pull/5/files#diff-3 (26)
- M src/cd++/utils/strutil.h https://github.com/SimulationEverywhere/CDPP_ExtendedStates-codename-Santi/pull/5/files#diff-4 (7)
Patch Links:
- https://github.com/SimulationEverywhere/CDPP_ExtendedStates-codename-Santi/pull/5.patch
https://github.com/SimulationEverywhere/CDPP_ExtendedStates-codename-Santi/pull/5.diff
— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/SimulationEverywhere/CDPP_ExtendedStates-codename-Santi/pull/5, or mute the thread https://github.com/notifications/unsubscribe-auth/AHhmjribaInIwIYWq0mYNqpn4wHa1oJIks5uwt5BgaJpZM4Ypebz .
thepalbi
Some imports are placed differently inside MacOS C standard library, like hashmap. Also, since the default C++ compiles is Clang, some warning are raised as errors. A fix is provided for them.