PDA

View Full Version : Downloadable Multi-Edit Rules Peg CPU @ 100%


18-Mar-2004, 06:56 PM
I downloaded the Multi-Edit XML file since I'm currently involved in a CMAC project. Whenever I compare .S files, the CPU pegs @ 100%.

Some experimenting seems to indicate that the problem lies in the first regular expression for line weights. If I delete it, performance is normal.

I'm going to spend some time with the regex when I get a few minutes, but I figured I'd pass this along. The expression is:

^[ \t\r\xFF]*([A-Za-z_$][A-Za-z0-9_$]*)+[ \t\r\xFF]+[A-Za-z_$][A-Za-z0-9_$]*\(

Craig
23-Mar-2004, 09:04 AM
Thanks for the bug report. These were contributed rules, so we haven't tested them, but I've emailed the original contributor, so we should have an updated set fairly soon.

02-Jun-2005, 12:46 PM
Was there ever any response on this? I just tried a Multi-Edit file compare and ran into this same problem.

wayneo
02-Jun-2005, 12:50 PM
That was me, actually.

Craig
02-Jun-2005, 01:00 PM
If you send in a sample of the files that this is having trouble with I can take a look at it. Just guessing based on the regular expression I'd say the correct expression would be this:

^[ \t\r\xFF]*[A-Za-z_$][A-Za-z0-9_$]*[ \t\r\xFF]+[A-Za-z_$][A-Za-z0-9_$]*\(

Craig
02-Jun-2005, 03:39 PM
We did get a reply from the original author, and it must have slipped through the cracks. He said that he's gone back to the standard "C" rules, and I think that's the best thing to do, given the circumstances. The regular expressions that are causing the problem are just for line weights, and I don't think deleting them will significantly affect the alignment. So, if you've downloaded the Multi-Edit rules and are having trouble, just delete them and add "*.sh;*.s;*.db" to your "C/C++ Source" rules. We'll pull the Multi-Edit rules from the website later today.

wayneo
02-Jun-2005, 04:11 PM
Thanks for the work.

By the way, the .db files are binary files and seem to be handled well by the rule "Multi-Edit Database". But yes, .s and .sh files are handled fine using the C/C++/C## Source rules.