Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

richtige g++ optionen setzten #74

Open
SinForest opened this issue Sep 28, 2017 · 3 comments
Open

richtige g++ optionen setzten #74

SinForest opened this issue Sep 28, 2017 · 3 comments
Assignees

Comments

@SinForest
Copy link
Contributor

g++ macht weirde Dinge (Stichwort: kein return in non-void functions), darum sollten wir auf (das sowieso viel bessere) clang++ umsteigen.

Da der Befehl so lang ist, könnten wir den Studis auch gleich noch einen Alias anlegen (nicht 'c++', das verwirrt nur!!).

Im Skript muss dazu vmtl fast nichts geändert werden.

@friedz
Copy link
Member

friedz commented Sep 30, 2017

ich würde keinen alias für sie anlegen wir können aber die .bashrc erwähnen und sagen schreiben wie da aliase rein geschrieben werden

@christian-heusel
Copy link
Member

Ich würde für dieses Jahr mal einen Alias, der g++="g++ -Wall -Werror -Wextra" gleichsetzt ausprobieren und schauen ob ein wechsel auf clang dann noch nötig ist...
Ich lasse das Issue mal noch offen bis ich die Rechner eingerichtet habe 😊

Für das von dir beschriebene Beispiel schenkt sich das dann nämlich nichts mehr:

$ g++ test.cpp -o test
test.cpp: In function ‘int add(int, int)’:
test.cpp:2:7: warning: statement has no effect [-Wunused-value]
    2 |     a + b;
      |     ~~^~~
test.cpp:3:1: warning: no return statement in function returning non-void [-Wreturn-type]
    3 | }
      | ^
$ clang++ test.cpp
test.cpp:2:7: warning: expression result unused [-Wunused-value]
    a + b;
    ~ ^ ~
test.cpp:3:1: warning: control reaches end of non-void function [-Wreturn-type]
}
^
2 warnings generated.

./test.cpp:

int add(int a, int b){
    a + b;
}
int main(int argc, char *argv[]) {
    add(1,2);
    return 0;
}

@christian-heusel christian-heusel self-assigned this Aug 10, 2019
@christian-heusel christian-heusel changed the title Clang++ statt G++ richtige g++ optionen setzten Aug 10, 2019
@jenuk
Copy link
Contributor

jenuk commented Aug 10, 2019

Dann solltest du hier darauf achten, dass das Kapitel über Warnings (aktuell 16) noch Sinn ergibt

@calvalre calvalre removed their assignment Oct 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants