Rémi Bernon (@rbernon) commented about dlls/dmime/miditracks.c:
+}
+static HRESULT WINAPI midi_sequence_track_RemoveNotificationType(
- IDirectMusicTrack8 *iface, REFGUID notiftype)
+{
- struct midi_sequence_track *This = impl_from_IDirectMusicTrack8(iface);
- TRACE("(%p, %s): method not implemented\n", This,
debugstr_dmguid(notiftype));- return E_NOTIMPL;
+}
+static HRESULT WINAPI midi_sequence_track_Clone(IDirectMusicTrack8 *iface,
MUSIC_TIME mtStart,MUSIC_TIME mtEnd,IDirectMusicTrack **ppTrack)
I'm going to nitpick again but let's start with it so that we can get it out of the way:
1) I find the wrapping used here unusual and not very readable. We have some kind of flexible line length limit, not a very hard 80 char limit like seems to be used here.
2) Long arguments lines are more often wrapped instead of having each arguments on their line.
2) Don't use hungarian notation even if the documented interface method signature does, use snake case and prefer separating words with underscores.
Because of how it looks I'm assuming this was formatted with clang-format, which is okay if that helps, but it's still not going to save you from manually checking for readability and consistency with the rest of the module code.
Tweaking the config will help getting nicer results I think. Here's the configuration I've used for dmusic, which is more or less a WineD3D-like code style module (no space in parentheses, double indent continuation):
``` --- DisableFormat: false SortIncludes: false Language: Cpp IndentWidth: 4 ColumnLimit: 100 BreakBeforeBraces: Allman AlignAfterOpenBracket: DontAlign AllowAllArgumentsOnNextLine: true AllowShortIfStatementsOnASingleLine: AllIfsAndElse AllowShortCaseLabelsOnASingleLine: true AllowShortFunctionsOnASingleLine: None AllowShortLoopsOnASingleLine: true ContinuationIndentWidth: 8 BinPackArguments: true BinPackParameters: true PenaltyBreakString: 1000 PenaltyExcessCharacter: 1 PenaltyIndentedWhitespace: 1 PenaltyBreakAssignment: 10000 PenaltyBreakOpenParenthesis: 10000 PenaltyReturnTypeOnItsOwnLine: 10000 PenaltyBreakBeforeFirstCallParameter: 10000 ... ```
Note that even with this configuration I don't believe clang-format does a super good job, and there's still several places where reformatting the whole file will need manual adjustments.