Skip to content
New issue

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

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

Already on GitHub? # to your account

AppMakers hazi tanulasg #82

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kemakee
Copy link
Contributor

@kemakee kemakee commented Dec 15, 2019

No description provided.

Copy link
Contributor

@csorbakristof csorbakristof left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Két apró változást kérek, hogy egységes legyen...


A Qt programunk futtatásakor rögtön az elején SIGSEGV-t dobott, még meg sem jelent semmi. Debugger csak annyit mondott, hogy az alkalmazás .exec(); -ig jut el, a többi a qt library-n belül van, ahonnan a jogosulatlan memória hivatkozás történt. [Stackoverflow](https://stackoverflow.com/questions/47330449/qt-segmentation-fault-at-exec) segített végül: a QApplication construktorának az argc referencia szerint kell (ez valóban le van írva a [dokumentációjában](https://doc.qt.io/qt-5/qapplication.html#QApplication), viszont a [példaprogramban](https://github.com/csorbakristof/alkalmazasfejlesztes/blob/af964ed7e9bf1a8f6c1b1838eb39d9a5491e4f6c/SimpleTelemetryVisualizer/main.cpp#L7) nem így volt).

QApplication::QApplication(int &argc, char **argv)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Forráskódnál használjátok a ``` környezetet, hogy legyen syntax highlighting.

@@ -0,0 +1,36 @@
# AppMakers csapat tanulságai #
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A fájl elejéről hiányzik a fejléc, meg a könyvtár neve sem stimmel. Legyen a tiétek a 46-os. A fejlécben a tags legyen "alkfejl afhf skipfromindex".

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants