[ cobertura-Patches-3304996 ] Don't try to color non-Java code by syntax

[SourceForge.net <no...@so...>]

Patches item #3304996, was opened at 2011-05-20 12:26
Message generated for change (Tracker Item Submitted) made by jouni
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=720017&aid=3304996&group_id=130558

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Jouni K Seppnen (jouni)
Assigned to: Nobody/Anonymous (nobody)
Summary: Don't try to color non-Java code by syntax

Initial Comment:
(This addresses partially the same problem as #3304991 in a different way, but IMHO both are worth applying.)

Since Cobertura is implemented via instrumenting class files, it is to some degree usable for JVM languages other than Java. However, applying Java code colorization to non-Java code will result in nonsensical output. It is better to view e.g. Clojure source without any colorization than with mistaken colorization. This patch looks at the filename of the source file, and if it does not end in .java, does not attempt to apply colorization.

The copyright to this patch belongs to my employer, ZenRobotics Ltd., and I have received permission to submit the patch to Cobertura. The patch is against Subversion branch v1_9_4.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=720017&aid=3304996&group_id=130558