<div><span style="color: rgb(160, 160, 168); ">On Thursday, August 16, 2012 at 6:00 AM, vlc-devel-request@videolan.org wrote:</span></div><blockquote type="cite" style="border-left-style:solid;border-width:1px;margin-left:0px;padding-left:10px;"><span><div>Brendon,</div><div><br></div><div>Thanks for the patch!</div><div><br></div><div>On 16.08.2012, at 03:34, Brendon Justin <<a href="mailto:brendonjustin@gmail.com">brendonjustin@gmail.com</a>> wrote:</div><div><br></div><blockquote type="cite"><div><div><br></div><div>+- (IBAction)prev:(id)sender;</div><div>+- (IBAction)backward:(id)sender;</div><div><br></div><div>+- (IBAction)next:(id)sender;</div><div>+- (IBAction)forward:(id)sender;</div></div></blockquote><div><br></div><div>These don't belong here but in MainWindow.h to ensure compatibility with Xcode 3.2's interface builder. Support for IBActions in categories was added in Xcode 4.2 as far as I remember.</div><div><br></div><div>Btw. Which compiler is so picky about this? Obj-C tolerates private methods without previous declaration just fine, since the dynamic lookup doesn't care about declarations, but about actually implemented methods only.</div><div><br></div><div><br></div><div>Best regards,</div><div><br></div><div>Felix</div></span></blockquote><div><div>Hi Felix,</div><div><br></div><div>A handful of functions were used before they were declared, I think the add/removeJumpButtons and add/removePlaymodeButtons functions.  Most of the functions were not used before their definitions and did not cause warnings though.  I thought that doing all of the undeclared functions would be better than just the ones that caused warnings, thus the long list.</div><div><br></div><div>I'll resend the patch with the IBActions moved to the header file in a bit.</div><div><br></div><div>As an aside, having different methods named fwd: and forward: (and bwd: and backward:) can't be good for clarity.</div><div><div><br></div><div>Thanks,</div><div>Brendon Justin</div></div>
                </div>