Darryl Okahata <darr...@sonic.net> writes: > Can I get a patch for this? I'd like to see what's changed, and I don't > see a patch in the cscope tickets. > > (As long as there isn't unnecessary code duplication, I should be able > to merge it pretty quickly.) > > Thanks.
Hi Darryl. A diff between the 15.8a release and the latest code is here: http://pub.secretsauce.net/xcscope.el.2013_09_16.patch However I ask that you do not commit it. This patch is large, and is made up of many discrete commits in version control. There is value in preserving the history, so if the changes look good to you, the git repo should be imported rather than squashing it all into a single patch. The history is here for easy review: https://github.com/dkogan/cscope/commits/xcscope_patches[1] If we decide that leaving xcscope.el as part of the larger cscope repository makes sense then I can do the import into CVS. There was a question of coupled releases I asked in an earlier email. Also it would be great to get this code a bit more tested before doing a merge; if you can try it out, that'd be great. As I mentioned to you earlier, I only tested this in emacs 24. What is the oldest version of xemacs you were targeting? How about GNU emacs? I'll test these once I know what they are. Thanks. dima ------------------------------------------------------------------------------ LIMITED TIME SALE - Full Year of Microsoft Training For Just $49.99! 1,500+ hours of tutorials including VisualStudio 2012, Windows 8, SharePoint 2013, SQL 2012, MVC 4, more. BEST VALUE: New Multi-Library Power Pack includes Mobile, Cloud, Java, and UX Design. Lowest price ever! Ends 9/20/13. http://pubads.g.doubleclick.net/gampad/clk?id=58041151&iu=/4140/ostg.clktrk _______________________________________________ Cscope-devel mailing list Cscope-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/cscope-devel
