The following document contains the listing of user tags found in the code. Below is the summary of the occurrences per tag.
| Tag | Total number of occurrences |
|---|---|
| FIXME | 26 |
| TODO | 16 |
Each tag is detailed below:
Number of occurrences found in the code: 26
| org.sourceforge.xradar.AbstractProcess | Line |
|---|---|
| This exception catching is ugly !!!!!! | 183 |
| this so ugly, it actually requires an other fixme ! Only if we run into this error we should not take care of the exception | 192 |
| org.sourceforge.xradar.DefaultReportValues | Line |
| Adding url into property produces this exception the following code is a workaround | 24 |
| org.sourceforge.xradar.XRadarException | Line |
| is this really the good way to do it ? | 9 |
| org.sourceforge.xradar.XRadarStaticsCmdLine | Line |
| this is ugly ! | 127 |
| org.sourceforge.xradar.XSLTMerger | Line |
| using DOMSource here may (again) lead to performance issue on large project | 105 |
| org.sourceforge.xradar.ant.RadarLoaderTask | Line |
| This exception catching is ugly !!!!!! | 63 |
| Should be moved inside AbstractProcess | 64 |
| this so ugly, it actually requires an other fixme ! Only if we run into this error we should not take care of the exception | 74 |
| org.sourceforge.xradar.ant.StaticsTask | Line |
| externalize this xml/api? | 111 |
| org.sourceforge.xradar.dynamics.BasicRunScenario | Line |
| Not ready yet ! | 76 |
| org.sourceforge.xradar.dynamics.Dynamics | Line |
| this is not really elegant | 107 |
| org.sourceforge.xradar.resolver.XRadarResourceLoader | Line |
| not really sure this OS checking code is still usefull | 32 |
| org.sourceforge.xradar.statics.BasicRunScenario | Line |
| Not ready yet , simply missing some appropriate classes files... | 47 |
| this is convenient but no really pertinent ! | 80 |
| this is convenient but no really pertinent ! | 105 |
| this is convenient but no really pertinent ! | 132 |
| this is convenient but no really pertinent ! | 161 |
| this is convenient but no really pertinent ! | 192 |
| org.sourceforge.xradar.statics.MergeReportEngine | Line |
| Add parameters to message | 141 |
| Add parameters to message | 155 |
| Add arguments to message | 173 |
| org.sourceforge.xradar.statics.renderers.XslHtmlWebsite | Line |
| Find a better way than tweak XPathAnalyser into this... | 110 |
| org.sourceforge.xradar.util.FileUtils | Line |
| this class should have a logger and log this kind of debug info: System.out.println(">>> index:" + filename.lastIndexOf(FileUtils.FILE_SEPARATOR) + "[separator:" + FileUtils.FILE_SEPARATOR + "]"); System.out.println(">>> called with" + filename); | 79 |
| Repair this hack | 123 |
| org.sourceforge.xradar.util.StreamUtils | Line |
| move to calling this following line to any calling code logger.log(Level.FINE,"Retrieving filename from URL:" + filename); | 128 |
Number of occurrences found in the code: 16
| org.sourceforge.xradar.AbstractProcess | Line |
|---|---|
| Add some test ( dir exits ? writable ?) File dir = new File (docsHome) if (dir.exists() && dir.canWrite()){ | 119 |
| Maybe simply use the setURIResolver on the transformer ? | 184 |
| org.sourceforge.xradar.ant.DynamicsTask | Line |
| Check that directory exists and is readable ! | 194 |
| Also check for an index.html file to assert if the report was successfully generated ? | 195 |
| org.sourceforge.xradar.config.ReleaseDataLoader | Line |
| Don't know if xradar should fail or use default values | 66 |
| Move to a static lib ? | 107 |
| org.sourceforge.xradar.statics.MergeReportEngine | Line |
| refactor this to have the "File handling" part of the code relying more on FileUtils. | 137 |
| Still a StringWriter here, maybe still source of memory consumption issue ? | 234 |
| Add success/failure checking, semantic-based (Look for specific xml part that should have been added by the merge ... Maybe using the Analyser API ? | 246 |
| To some kind of singleton for this ? | 277 |
| org.sourceforge.xradar.statics.Statics | Line |
| I don't really remember why this next part is necessary... May be it no longer required as we fix stuff about win32 path at some other place | 151 |
| Regroup those two parameter in a dedicated POJO to get a better signature | 177 |
| org.sourceforge.xradar.statics.renderers.XslHtmlWebsite | Line |
| Replace the distance graph from 'old style' by the new one ! | 97 |
| org.sourceforge.xradar.util.FileUtils | Line |
| Is this value the more relevant ? | 136 |
| Check for write access ? | 187 |
| org.sourceforge.xradar.util.StreamUtils | Line |
| Maybe a good candidate to create java5 static imports | 31 |