-
Notifications
You must be signed in to change notification settings - Fork 0
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
Added framebuffer class and documentation #3
Conversation
7b0c752
to
45a7ad2
Compare
45a7ad2
to
a4ae015
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good and reads better. See comments. Also good idea on starting to move include guards away from #pragma once
; added a commit to do the rest too.
5d423bf
to
e3271f6
Compare
e3271f6
to
092d782
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry looks like I forgot to resubmit a review. One comment from last time was not addressed and please remove your usage of [feature]
and [refactor]
naming on the PRs. The verb usage in the title implies this and we can decide on a set of labels in the future if we really need to sort PRs down the line.
b77cdf6
to
ea6696c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. One last piece of feedback but good to merge now once you do that.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry one last comment
39188e3
to
4c7217d
Compare
4c7217d
to
0a4aa7f
Compare
Added the Framebuffer class. This class abstracts the Vulkan
Framebuffer
object and makes it a little more managable/readable when using it.