Resolve "Resource Management"

Closes #22 (closed)

added 👍 Feature 💻 Doing + 1 deleted label

added 1 commit

• 1cbb3910 - [#22 (closed)] Adds beginnings template buffer class

Compare with previous version

added 1 commit

• c174222a - [#22 (closed)] Bugfixes in Constructors

Compare with previous version

added 1 commit

• 1a2ca2d0 - [#22 (closed)] Minor Correction in documentation

Compare with previous version

Was noch unklar / zu tun ist:

• Einbindung in Core, -> Handle wird erwartet, soll aber ersetzt werden
• Erweiterung des Handle-Structs innerhalb der Klasse (für die Zukunft)
• bei der Fill Methode werden Daten übergeben, ich bin nicht sicher, welchen Typ dann das Argument haben muss und wie die fill methode dann aufgerufen werden soll

Die Funktionalität der eigentlichen Klasse sollte ansonsten stehen.

added 1 commit

• 054ee07c - [#22 (closed)] Made some adjustments and added comments

Compare with previous version

added 1 commit

• 93f66c38 - [#22 (closed)] Added createBuffer()-template-function to core. Uses buffer instead of handle for now

Compare with previous version

added 1 commit

• 911ff109 - [#22 (closed)] Minor syntax-fix

Compare with previous version

added 1 commit

• 7f5c76a8 - [#22 (closed)] Changed create function in core

Compare with previous version

Hast du schon versucht mit develop zu mergen (also den aktuellen Stand von develop hier in die Branch rein, um später Konflikten vorzubeugen beim eigentlichen mergen)? Könnte ich ansonsten auch übernehmen...

added 1 commit

• 362702b3 - Added create()-function for buffer to allow the code to handle issues

Compare with previous version

Ich habe mal den Code aus dem Constructor bewegt. Das erzeugen in der main.cpp klappt jetzt auch ohne std::move(), da der Compiler implizit moved aufgrund fehlenden Kopierens. Die MemoryRequirements habe ich auch schonmal aus der Klasse genommen. Die können direkt vom Buffer abgefragt werden eigentlich (tatsächlich wird hauptsächlich die Size abgefragt, welche ja auch noch separat gespeichert wird).

Nein, ich hatte noch nicht versucht mit develop zu mergen. Kannst du gerne übernehmen, ich wollte/würde das ansonsten morgen machen. Ich war grade dabei schon mal Funktionen in die CPP auszulagern, habe da aber deinen Commit nicht mitbekommen, d.h. ich schaue das jetzt mal an und passe das was ich gemacht habe an und comitte das dann gleich hier

added 1 commit

• ae06cc85 - [#22 (closed)] Outsourced some functionality from hpp to cpp

Compare with previous version

Ich bin die .hpp durchgegangen und habe alles, was nichts mit T zu tun hat (und länger war als eine Zeile) versucht in die CPP zu schieben. Resultierte Änderungen:

• Dekonstruktor(nicht sicher ob sich das so lohnt)
• den Switch Case in der create Funktion
• das Allokieren samt hässlicher Hilffunktion

Frage die ich mir eben gestellt habe: Kann man in der Fill() Methode die Map und Unmap Methode benutzen? Vorher stand dort das selbe drin, nun mit den neuen Casts und return Werten bin ich mir nicht sicher.

Jup, die Funktionen klappen soweit. Ich denke, dass kann so bereits auf develop gemerged werden. Dann können die folgenden Issues und Branches wieder von develop ausgehen.

Gibt es einen speziellen Grund wieso Teile der Implementierung im Header gelandet sind? Die Begründung hier im Thread verstehe ich nicht.

Weil es Templates sind, die gehen nur im Header.

Oh, hehe, I see ^^' Danke!

approved this merge request

marked this merge request as ready

resolved all threads

added 106 commits

• ae06cc85...81bfa2d9 - 105 commits from branch develop
• 73b27493 - Merge branch 'develop' into '22-resource-management'

Compare with previous version

mentioned in commit 07f72bd7