Unused additional functions in 14_command_buffers

[asuessenbach]

In 14_command_buffers.cpp, there are (besides other stuff) two new functions that are not used: recordCommandBuffer and transition_image_layout.

• Is that intentional? Or should they be introduced with 15_hello_triangle, where they are actually used?
• If they should be kept in 14_command_buffers, should the synchronization2 feature be requested as well? That would be needed if those functions would be used.
• In order to keep naming scheme consistency, should transition_image_layout be renamed to transitionImageLayout?

[SaschaWillems]

In order to keep naming scheme consistency, should transition_image_layout be renamed to transitionImageLayout?

Yes, naming should be consistent with the rest of the tutorial, which is lower pascal case. Also see #153

[asuessenbach]

Yes, naming should be consistent with the rest of the tutorial, which is lower pascal case.

Well, not exactly. Most functions I've seen here use the camelCase convention.

[SaschaWillems]

That's what I meant. lower pascal case is actually camelCase ;)

[asuessenbach]

Stays the question whether those two functions should materialize in chapter 14, where they are not used, or in chapter 15.

[SaschaWillems]

I'd go for the latter. Having functions in a chapter where they aren't used might be confusing.