<div dir="ltr"><div>S<span style="line-height:1.5">ince yesterday was a rainy day over here, I took some time at the unimplemented dce6 functions and</span><span class="inbox-inbox-inbox-inbox-Apple-converted-space" style="line-height:1.5">  </span><span style="line-height:1.5">I'm almost done with them.</span></div><div><br></div><div>I still need to:</div><div>- mimic how MASKs and SHIFTs are used in dce_v8_0.c (purely for consistency and clarity)</div><div>-- rename some variables and split imbriqued MASKs and SHIFTs (almost completed for dce_v6_0.c usages and some other files that were using the same defined variables)</div><div>-- move MASKs and SHIFTs to their own file (dce_6_0_sh_mask.h)</div><div>- complete implementation of two remainging functions:</div><div>--- <span style="font-size:13.3333px;line-height:normal">dce_v6_0_afmt_update_ACR()</span></div><div><span style="font-size:13.3333px;line-height:normal">--- dce_v6_0_afmt_update_avi_infoframe()</span></div><div><br></div><div>I found a few duplicated "#define" in sid.h while porting code that I'm cleaning at the same time.</div><div><br></div><div>So, in the next few days, I'll send patches for cleaning duplicated "#define" in sid.h, renaming, splitting and moving MASKs and SHIFTs to their own file first. It could be grouped as follow: a patch for removed duplicates, one for renaming and splitting masks and shifts, and a last one for moving them to their own file.<span style="line-height:1.5"> </span></div><div><br></div><div>However, after that, would it be better to send a single patch for all unimplemented functions or should I split them? Per function (or group of functions)?</div><div><br></div><div>Thanks for your input.</div><div><span style="line-height:1.5">Alexandre Demers</span><br></div></div>