Menu
▾
▴
jtidy-devel — JTidy mailing list for developers
You can subscribe to this list here.
| 2004 |
Jan
(29) |
Feb
(1) |
Mar
(6) |
Apr
(31) |
May
(2) |
Jun
(2) |
Jul
(13) |
Aug
(31) |
Sep
(41) |
Oct
(12) |
Nov
(13) |
Dec
(4) |
|---|---|---|---|---|---|---|---|---|---|---|---|---|
| 2005 |
Jan
(17) |
Feb
(3) |
Mar
(3) |
Apr
|
May
(1) |
Jun
(2) |
Jul
(1) |
Aug
(3) |
Sep
(3) |
Oct
(1) |
Nov
(2) |
Dec
(6) |
| 2006 |
Jan
(4) |
Feb
(6) |
Mar
(2) |
Apr
(1) |
May
|
Jun
|
Jul
(21) |
Aug
(7) |
Sep
(5) |
Oct
(4) |
Nov
(2) |
Dec
(2) |
| 2007 |
Jan
(1) |
Feb
|
Mar
|
Apr
(2) |
May
|
Jun
|
Jul
(1) |
Aug
(2) |
Sep
(2) |
Oct
(2) |
Nov
|
Dec
(1) |
| 2008 |
Jan
(1) |
Feb
(1) |
Mar
(7) |
Apr
(2) |
May
|
Jun
|
Jul
(1) |
Aug
|
Sep
(1) |
Oct
(1) |
Nov
(2) |
Dec
(8) |
| 2009 |
Jan
(1) |
Feb
|
Mar
|
Apr
|
May
(2) |
Jun
(2) |
Jul
(5) |
Aug
(24) |
Sep
(16) |
Oct
(8) |
Nov
(42) |
Dec
(3) |
| 2010 |
Jan
(8) |
Feb
(8) |
Mar
(14) |
Apr
(29) |
May
(2) |
Jun
(1) |
Jul
(11) |
Aug
(47) |
Sep
(4) |
Oct
(16) |
Nov
(18) |
Dec
|
| 2011 |
Jan
(5) |
Feb
(4) |
Mar
(2) |
Apr
|
May
|
Jun
(10) |
Jul
(50) |
Aug
(4) |
Sep
(4) |
Oct
(1) |
Nov
(4) |
Dec
|
| 2012 |
Jan
|
Feb
|
Mar
|
Apr
|
May
(2) |
Jun
(8) |
Jul
|
Aug
|
Sep
|
Oct
|
Nov
|
Dec
|
| 2017 |
Jan
|
Feb
|
Mar
|
Apr
|
May
|
Jun
|
Jul
|
Aug
(1) |
Sep
|
Oct
|
Nov
|
Dec
|
|
From: SourceForge.net <no...@so...> - 2010-08-17 00:51:49
|
Bugs item #3037011, was opened at 2010-07-30 16:50 Message generated for change (Comment added) made by aditsu You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=113153&aid=3037011&group_id=13153 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: Closed >Resolution: Accepted Priority: 5 Private: No Submitted By: Johannes Weberhofer (weberho) >Assigned to: Adrian Sandor (aditsu) Summary: Patch: JTidyTask.java.patch Initial Comment: This patch fixes the check for a valid properies-file ---------------------------------------------------------------------- >Comment By: Adrian Sandor (aditsu) Date: 2010-08-17 08:51 Message: Applied; btw, the removal of whitespace at the end of comment lines is a bit distracting, could you focus on the code changes? Thanks ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=113153&aid=3037011&group_id=13153 |
|
From: SourceForge.net <no...@so...> - 2010-08-17 00:36:13
|
Bugs item #3038697, was opened at 2010-08-03 16:48 Message generated for change (Comment added) made by aditsu You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=113153&aid=3038697&group_id=13153 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: Johannes Weberhofer (weberho) Assigned to: Adrian Sandor (aditsu) Summary: Remove lots of the reasons for failing junit-tests Initial Comment: Most junit tests were failing because there were jtidy-version information in the resulting messages definied. I have fixed those issues plus some general issues, so that there should be 'only' around 90 failing junit tests at the end. ---------------------------------------------------------------------- >Comment By: Adrian Sandor (aditsu) Date: 2010-08-17 08:36 Message: Um.. did you create this patch just by running the tests and assuming the current output is correct? If so, why do you think it is good to make that assumption? ---------------------------------------------------------------------- Comment By: Johannes Weberhofer (weberho) Date: 2010-08-03 18:14 Message: Ok. I'll send you a patch so that maven produces 1.4 code,too (in trunk) and have a look on the new sources. ---------------------------------------------------------------------- Comment By: Adrian Sandor (aditsu) Date: 2010-08-03 17:43 Message: I don't use maven so I can't comment on the pom, but afaik, the DOM part of the java api was added in 1.4, so JTidy couldn't run in 1.3 (or maybe only part of it). Also, recent versions of JTidy were compiled for 1.4. Yes, that branch is usable and I think better than trunk at this point, but I'm not sure about organizing the new classes, maybe you can take a look and give some feedback. The new test suite is TidyTests. Thanks for your contributions. ---------------------------------------------------------------------- Comment By: Johannes Weberhofer (weberho) Date: 2010-08-03 17:28 Message: Java compatibility is set to 1.3 in the pom.xml in trunk. I think, it whould be set to 1.4, as it 1.3 is outdated for a long time. Is the current CodeUpdateAndJava5 branch useable? Then I would possibly to switch to that branch and help improving jtidy (as my time allows me so). ---------------------------------------------------------------------- Comment By: Adrian Sandor (aditsu) Date: 2010-08-03 17:07 Message: I kind of gave up on those test cases, and made a new test suite on the CodeUpdateAndJava5 branch, where "only" 73 tests fail now (meaning the result is not 100% byte-by-byte identical with Tidy). But I think it's good to fix these tests too, especially if we plan to maintain the trunk version (compatible with java 1.4) longer. I'll check your patch when I find some time. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=113153&aid=3038697&group_id=13153 |
|
From: SourceForge.net <no...@so...> - 2010-08-17 00:25:39
|
Bugs item #3038314, was opened at 2010-08-02 22:50 Message generated for change (Comment added) made by aditsu You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=113153&aid=3038314&group_id=13153 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: Johannes Weberhofer (weberho) Assigned to: Adrian Sandor (aditsu) Summary: encoding of last "-" character is not working correctly Initial Comment: in the test's case the last '-' character at the end of the paragraph is not encoded correctly. ---------------------------------------------------------------------- >Comment By: Adrian Sandor (aditsu) Date: 2010-08-17 08:25 Message: Hi, could you try the attached patch please? It seems to fix the problem but I wonder if it breaks anything. Btw, it doesn't seem to break any more tests on the java5 branch. ---------------------------------------------------------------------- Comment By: Johannes Weberhofer (weberho) Date: 2010-08-03 01:22 Message: I'm using trunk. I have seen the problem for at least two years but never filed a bug. ---------------------------------------------------------------------- Comment By: Adrian Sandor (aditsu) Date: 2010-08-03 01:05 Message: What version of jtidy did you use and what program/command did you run? ---------------------------------------------------------------------- Comment By: Johannes Weberhofer (weberho) Date: 2010-08-03 00:46 Message: I have attached jtidies' output. Have a look at the "incluindo – mas" section. I have tried to shorten the problematic sequence as short as I can. The strange thing is that - when you remove some things from the beginning, everything works nicely. ---------------------------------------------------------------------- Comment By: Adrian Sandor (aditsu) Date: 2010-08-02 23:10 Message: I opened both the input and output files in a browser and they look identical. Even the html source looks identical except for different line wrapping. What exactly is the problem? ---------------------------------------------------------------------- Comment By: Johannes Weberhofer (weberho) Date: 2010-08-02 22:58 Message: the '-' is a unicode character, not a regular - sign. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=113153&aid=3038314&group_id=13153 |
|
From: SourceForge.net <no...@so...> - 2010-08-16 23:28:57
|
Bugs item #3038730, was opened at 2010-08-03 18:17 Message generated for change (Settings changed) made by aditsu You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=113153&aid=3038730&group_id=13153 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: Johannes Weberhofer (weberho) >Assigned to: Adrian Sandor (aditsu) Summary: patch: fixed maven warnings in trunk; produce java 1.4 code Initial Comment: This patch fixes the maven file, as it includes version definitions for the different plugins and produces java 1.4 code instead of 1.3 code... ---------------------------------------------------------------------- >Comment By: Adrian Sandor (aditsu) Date: 2010-08-17 07:28 Message: Hi, I'm having some problems with the patch, could you please attach the complete pom.xml file? ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=113153&aid=3038730&group_id=13153 |
|
From: SourceForge.net <no...@so...> - 2010-08-13 21:22:23
|
Bugs item #3025841, was opened at 2010-07-06 15:22 Message generated for change (Comment added) made by weberho You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=113153&aid=3025841&group_id=13153 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: Tidy functionality Group: None Status: Closed Resolution: Invalid Priority: 5 Private: No Submitted By: Dirk Przybylla (dpr42) Assigned to: Adrian Sandor (aditsu) Summary: unescaped & or unknown entity in URL Initial Comment: Hello, i implemented some links The eclipsetidy version 1.2.2 marks the following line after a check <a href="http://cds.sun.com/is-bin/INTERSHOP.enfinity/WFS/CDS-CDS_Developer-Site/en_US/-/USD/VerifyItem-Start/jre-6u20-windows-i586.exe?BundledLineItemUUID=2RKJ_hCyQ_oAAAEpg7o9a6NS&OrderID=pPSJ_hCyy5UAAAEpd7o9a6NS&ProductID=MUNIBe.pkFEAAAEns3oZAPYl&FileName=/jre-6u20-windows-i586.exe">Java Runzime Environment ab Version 1.5"</a> with Multiple annotations found at this line: - unescaped & or unknown entity "&ProductID" - column 222 - unescaped & or unknown entity "&OrderID" - column 189 - unescaped & or unknown entity "&FileName" - column 257 However, in this case the ampersand is the separator of the arguments of an Url and not an unescaped & or an unknown entity With best regards Dirk ---------------------------------------------------------------------- Comment By: Johannes Weberhofer (weberho) Date: 2010-08-13 23:22 Message: Inside HTML & must be properly encoded as & The warning is correct. ---------------------------------------------------------------------- Comment By: Adrian Sandor (aditsu) Date: 2010-07-31 00:03 Message: First of all, this is not the eclipsetidy project. However, your problem is quite clear: ampersands in urls need to be escaped in html pages, see http://htmlhelp.com/tools/validator/problems.html#amp ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=113153&aid=3025841&group_id=13153 |
|
From: SourceForge.net <no...@so...> - 2010-08-12 10:04:52
|
Bugs item #3038802, was opened at 2010-08-03 15:38 Message generated for change (Comment added) made by weberho You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=113153&aid=3038802&group_id=13153 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: Johannes Weberhofer (weberho) Assigned to: Adrian Sandor (aditsu) Summary: patch jtidy2: remove dead code Initial Comment: Eclipse has detected some unused code in the CodeUpdatedAndJava5 branch. I have reviewed and removed that code. ---------------------------------------------------------------------- >Comment By: Johannes Weberhofer (weberho) Date: 2010-08-12 12:04 Message: Adrian, yes, the patches could also be applied to trunk. I have checked the Clean.java code, and am yuite sure, that after exiting the "while (node.is(TagId.P) && noMargins(node))" section, there will alwys be remaining nodes: The paragraphs are striped off, and newLineNodes are added instead. As the while-section is only reached when nodes exist, there must be nodes upon leaving, too. Hope I'm right... ---------------------------------------------------------------------- Comment By: Adrian Sandor (aditsu) Date: 2010-08-12 01:37 Message: Doesn't this also apply to trunk? Anyway, it doesn't affect much.. unless it uncovers bugs; I think the dead code in Clean.java needs some review. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=113153&aid=3038802&group_id=13153 |
|
From: SourceForge.net <no...@so...> - 2010-08-11 23:38:01
|
Bugs item #3038802, was opened at 2010-08-03 21:38 Message generated for change (Comment added) made by aditsu You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=113153&aid=3038802&group_id=13153 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: Johannes Weberhofer (weberho) >Assigned to: Adrian Sandor (aditsu) Summary: patch jtidy2: remove dead code Initial Comment: Eclipse has detected some unused code in the CodeUpdatedAndJava5 branch. I have reviewed and removed that code. ---------------------------------------------------------------------- >Comment By: Adrian Sandor (aditsu) Date: 2010-08-12 07:37 Message: Doesn't this also apply to trunk? Anyway, it doesn't affect much.. unless it uncovers bugs; I think the dead code in Clean.java needs some review. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=113153&aid=3038802&group_id=13153 |
|
From: SourceForge.net <no...@so...> - 2010-08-11 23:25:56
|
Bugs item #3038807, was opened at 2010-08-03 21:43 Message generated for change (Comment added) made by aditsu You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=113153&aid=3038807&group_id=13153 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: Closed >Resolution: Accepted Priority: 5 Private: No Submitted By: Johannes Weberhofer (weberho) >Assigned to: Adrian Sandor (aditsu) Summary: patch jtidy2: cast null to array of object Initial Comment: the arguments should be casted to Object[] in the CodeUpdateAndJava5 branch. ---------------------------------------------------------------------- >Comment By: Adrian Sandor (aditsu) Date: 2010-08-12 07:25 Message: Applied, thanks ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=113153&aid=3038807&group_id=13153 |
|
From: SourceForge.net <no...@so...> - 2010-08-09 12:10:39
|
Bugs item #3041886, was opened at 2010-08-09 16:10 Message generated for change (Tracker Item Submitted) made by udh-winger You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=113153&aid=3041886&group_id=13153 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: DOM Support Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: Vladislav Isenbaev (udh-winger) Assigned to: Nobody/Anonymous (nobody) Summary: replaceChild shows strange behavior Initial Comment: replaceChild of JTidy DOM behaves strangely. It is not equivalent to : insertBefore(newChild,oldChild); return removeChild(oldChild); ------------------- Tidy tidy = new Tidy(); tidy.setXmlOut(true); Document doc = Tidy.createEmptyDocument(); Element html = doc.getDocumentElement(); Element a = doc.createElement("a"); Element b = doc.createElement("b"); a.appendChild(b); html.appendChild(a); tidy.pprint(doc, System.out); System.out.println("---------"); Element c = doc.createElement("c"); Element d = doc.createElement("d"); c.appendChild(d); html.replaceChild(c, a); tidy.pprint(doc, System.out); This code produces this output: <html> <a><b /> </a> </html> --------- <html> <c><b /> </c> </html> However, it should be: <html> <a><b /> </a> </html> --------- <html> <c><d /> </c> </html> ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=113153&aid=3041886&group_id=13153 |
|
From: SourceForge.net <no...@so...> - 2010-08-03 14:06:35
|
Bugs item #3038802, was opened at 2010-08-03 15:38 Message generated for change (Tracker Item Submitted) made by weberho You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=113153&aid=3038802&group_id=13153 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: Johannes Weberhofer (weberho) Assigned to: Nobody/Anonymous (nobody) Summary: patch jtidy2: remove dead code Initial Comment: Eclipse has detected some unused code in the CodeUpdatedAndJava5 branch. I have reviewed and removed that code. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=113153&aid=3038802&group_id=13153 |
|
From: SourceForge.net <no...@so...> - 2010-08-03 13:43:32
|
Bugs item #3038807, was opened at 2010-08-03 15:43 Message generated for change (Tracker Item Submitted) made by weberho You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=113153&aid=3038807&group_id=13153 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: Johannes Weberhofer (weberho) Assigned to: Nobody/Anonymous (nobody) Summary: patch jtidy2: cast null to array of object Initial Comment: the arguments should be casted to Object[] in the CodeUpdateAndJava5 branch. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=113153&aid=3038807&group_id=13153 |
|
From: SourceForge.net <no...@so...> - 2010-08-03 10:53:49
|
Bugs item #3036736, was opened at 2010-07-30 03:11 Message generated for change (Comment added) made by aditsu You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=113153&aid=3036736&group_id=13153 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: Closed >Resolution: Accepted Priority: 5 Private: No Submitted By: Johannes Weberhofer (weberho) >Assigned to: Adrian Sandor (aditsu) Summary: Patch: Force tests to run always with english locale Initial Comment: This patch forces usage of english error messages to match the expected english test-cases ---------------------------------------------------------------------- >Comment By: Adrian Sandor (aditsu) Date: 2010-08-03 18:53 Message: Applied, thanks ---------------------------------------------------------------------- Comment By: Johannes Weberhofer (weberho) Date: 2010-08-03 18:19 Message: without that patch, maven checks fails on non-english environments ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=113153&aid=3036736&group_id=13153 |
|
From: SourceForge.net <no...@so...> - 2010-08-03 10:19:13
|
Bugs item #3036736, was opened at 2010-07-29 21:11 Message generated for change (Comment added) made by weberho You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=113153&aid=3036736&group_id=13153 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: Johannes Weberhofer (weberho) Assigned to: Nobody/Anonymous (nobody) Summary: Patch: Force tests to run always with english locale Initial Comment: This patch forces usage of english error messages to match the expected english test-cases ---------------------------------------------------------------------- >Comment By: Johannes Weberhofer (weberho) Date: 2010-08-03 12:19 Message: without that patch, maven checks fails on non-english environments ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=113153&aid=3036736&group_id=13153 |
|
From: SourceForge.net <no...@so...> - 2010-08-03 10:17:04
|
Bugs item #3038730, was opened at 2010-08-03 12:17 Message generated for change (Tracker Item Submitted) made by weberho You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=113153&aid=3038730&group_id=13153 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: Johannes Weberhofer (weberho) Assigned to: Nobody/Anonymous (nobody) Summary: patch: fixed maven warnings in trunk; produce java 1.4 code Initial Comment: This patch fixes the maven file, as it includes version definitions for the different plugins and produces java 1.4 code instead of 1.3 code... ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=113153&aid=3038730&group_id=13153 |
|
From: SourceForge.net <no...@so...> - 2010-08-03 10:14:39
|
Bugs item #3038697, was opened at 2010-08-03 10:48 Message generated for change (Comment added) made by weberho You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=113153&aid=3038697&group_id=13153 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: Johannes Weberhofer (weberho) Assigned to: Adrian Sandor (aditsu) Summary: Remove lots of the reasons for failing junit-tests Initial Comment: Most junit tests were failing because there were jtidy-version information in the resulting messages definied. I have fixed those issues plus some general issues, so that there should be 'only' around 90 failing junit tests at the end. ---------------------------------------------------------------------- >Comment By: Johannes Weberhofer (weberho) Date: 2010-08-03 12:14 Message: Ok. I'll send you a patch so that maven produces 1.4 code,too (in trunk) and have a look on the new sources. ---------------------------------------------------------------------- Comment By: Adrian Sandor (aditsu) Date: 2010-08-03 11:43 Message: I don't use maven so I can't comment on the pom, but afaik, the DOM part of the java api was added in 1.4, so JTidy couldn't run in 1.3 (or maybe only part of it). Also, recent versions of JTidy were compiled for 1.4. Yes, that branch is usable and I think better than trunk at this point, but I'm not sure about organizing the new classes, maybe you can take a look and give some feedback. The new test suite is TidyTests. Thanks for your contributions. ---------------------------------------------------------------------- Comment By: Johannes Weberhofer (weberho) Date: 2010-08-03 11:28 Message: Java compatibility is set to 1.3 in the pom.xml in trunk. I think, it whould be set to 1.4, as it 1.3 is outdated for a long time. Is the current CodeUpdateAndJava5 branch useable? Then I would possibly to switch to that branch and help improving jtidy (as my time allows me so). ---------------------------------------------------------------------- Comment By: Adrian Sandor (aditsu) Date: 2010-08-03 11:07 Message: I kind of gave up on those test cases, and made a new test suite on the CodeUpdateAndJava5 branch, where "only" 73 tests fail now (meaning the result is not 100% byte-by-byte identical with Tidy). But I think it's good to fix these tests too, especially if we plan to maintain the trunk version (compatible with java 1.4) longer. I'll check your patch when I find some time. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=113153&aid=3038697&group_id=13153 |
|
From: SourceForge.net <no...@so...> - 2010-08-03 09:43:35
|
Bugs item #3038697, was opened at 2010-08-03 16:48 Message generated for change (Comment added) made by aditsu You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=113153&aid=3038697&group_id=13153 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: Johannes Weberhofer (weberho) Assigned to: Adrian Sandor (aditsu) Summary: Remove lots of the reasons for failing junit-tests Initial Comment: Most junit tests were failing because there were jtidy-version information in the resulting messages definied. I have fixed those issues plus some general issues, so that there should be 'only' around 90 failing junit tests at the end. ---------------------------------------------------------------------- >Comment By: Adrian Sandor (aditsu) Date: 2010-08-03 17:43 Message: I don't use maven so I can't comment on the pom, but afaik, the DOM part of the java api was added in 1.4, so JTidy couldn't run in 1.3 (or maybe only part of it). Also, recent versions of JTidy were compiled for 1.4. Yes, that branch is usable and I think better than trunk at this point, but I'm not sure about organizing the new classes, maybe you can take a look and give some feedback. The new test suite is TidyTests. Thanks for your contributions. ---------------------------------------------------------------------- Comment By: Johannes Weberhofer (weberho) Date: 2010-08-03 17:28 Message: Java compatibility is set to 1.3 in the pom.xml in trunk. I think, it whould be set to 1.4, as it 1.3 is outdated for a long time. Is the current CodeUpdateAndJava5 branch useable? Then I would possibly to switch to that branch and help improving jtidy (as my time allows me so). ---------------------------------------------------------------------- Comment By: Adrian Sandor (aditsu) Date: 2010-08-03 17:07 Message: I kind of gave up on those test cases, and made a new test suite on the CodeUpdateAndJava5 branch, where "only" 73 tests fail now (meaning the result is not 100% byte-by-byte identical with Tidy). But I think it's good to fix these tests too, especially if we plan to maintain the trunk version (compatible with java 1.4) longer. I'll check your patch when I find some time. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=113153&aid=3038697&group_id=13153 |
|
From: SourceForge.net <no...@so...> - 2010-08-03 09:28:30
|
Bugs item #3038697, was opened at 2010-08-03 10:48 Message generated for change (Comment added) made by weberho You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=113153&aid=3038697&group_id=13153 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: Johannes Weberhofer (weberho) Assigned to: Adrian Sandor (aditsu) Summary: Remove lots of the reasons for failing junit-tests Initial Comment: Most junit tests were failing because there were jtidy-version information in the resulting messages definied. I have fixed those issues plus some general issues, so that there should be 'only' around 90 failing junit tests at the end. ---------------------------------------------------------------------- >Comment By: Johannes Weberhofer (weberho) Date: 2010-08-03 11:28 Message: Java compatibility is set to 1.3 in the pom.xml in trunk. I think, it whould be set to 1.4, as it 1.3 is outdated for a long time. Is the current CodeUpdateAndJava5 branch useable? Then I would possibly to switch to that branch and help improving jtidy (as my time allows me so). ---------------------------------------------------------------------- Comment By: Adrian Sandor (aditsu) Date: 2010-08-03 11:07 Message: I kind of gave up on those test cases, and made a new test suite on the CodeUpdateAndJava5 branch, where "only" 73 tests fail now (meaning the result is not 100% byte-by-byte identical with Tidy). But I think it's good to fix these tests too, especially if we plan to maintain the trunk version (compatible with java 1.4) longer. I'll check your patch when I find some time. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=113153&aid=3038697&group_id=13153 |
|
From: SourceForge.net <no...@so...> - 2010-08-03 09:07:59
|
Bugs item #3038697, was opened at 2010-08-03 16:48 Message generated for change (Comment added) made by aditsu You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=113153&aid=3038697&group_id=13153 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: Johannes Weberhofer (weberho) >Assigned to: Adrian Sandor (aditsu) Summary: Remove lots of the reasons for failing junit-tests Initial Comment: Most junit tests were failing because there were jtidy-version information in the resulting messages definied. I have fixed those issues plus some general issues, so that there should be 'only' around 90 failing junit tests at the end. ---------------------------------------------------------------------- >Comment By: Adrian Sandor (aditsu) Date: 2010-08-03 17:07 Message: I kind of gave up on those test cases, and made a new test suite on the CodeUpdateAndJava5 branch, where "only" 73 tests fail now (meaning the result is not 100% byte-by-byte identical with Tidy). But I think it's good to fix these tests too, especially if we plan to maintain the trunk version (compatible with java 1.4) longer. I'll check your patch when I find some time. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=113153&aid=3038697&group_id=13153 |
|
From: SourceForge.net <no...@so...> - 2010-08-03 08:48:12
|
Bugs item #3038697, was opened at 2010-08-03 10:48 Message generated for change (Tracker Item Submitted) made by weberho You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=113153&aid=3038697&group_id=13153 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: Johannes Weberhofer (weberho) Assigned to: Nobody/Anonymous (nobody) Summary: Remove lots of the reasons for failing junit-tests Initial Comment: Most junit tests were failing because there were jtidy-version information in the resulting messages definied. I have fixed those issues plus some general issues, so that there should be 'only' around 90 failing junit tests at the end. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=113153&aid=3038697&group_id=13153 |
|
From: SourceForge.net <no...@so...> - 2010-08-02 17:22:11
|
Bugs item #3038314, was opened at 2010-08-02 16:50 Message generated for change (Comment added) made by weberho You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=113153&aid=3038314&group_id=13153 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: Johannes Weberhofer (weberho) Assigned to: Adrian Sandor (aditsu) Summary: encoding of last "-" character is not working correctly Initial Comment: in the test's case the last '-' character at the end of the paragraph is not encoded correctly. ---------------------------------------------------------------------- >Comment By: Johannes Weberhofer (weberho) Date: 2010-08-02 19:22 Message: I'm using trunk. I have seen the problem for at least two years but never filed a bug. ---------------------------------------------------------------------- Comment By: Adrian Sandor (aditsu) Date: 2010-08-02 19:05 Message: What version of jtidy did you use and what program/command did you run? ---------------------------------------------------------------------- Comment By: Johannes Weberhofer (weberho) Date: 2010-08-02 18:46 Message: I have attached jtidies' output. Have a look at the "incluindo – mas" section. I have tried to shorten the problematic sequence as short as I can. The strange thing is that - when you remove some things from the beginning, everything works nicely. ---------------------------------------------------------------------- Comment By: Adrian Sandor (aditsu) Date: 2010-08-02 17:10 Message: I opened both the input and output files in a browser and they look identical. Even the html source looks identical except for different line wrapping. What exactly is the problem? ---------------------------------------------------------------------- Comment By: Johannes Weberhofer (weberho) Date: 2010-08-02 16:58 Message: the '-' is a unicode character, not a regular - sign. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=113153&aid=3038314&group_id=13153 |
|
From: SourceForge.net <no...@so...> - 2010-08-02 17:05:32
|
Bugs item #3038314, was opened at 2010-08-02 22:50 Message generated for change (Comment added) made by aditsu You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=113153&aid=3038314&group_id=13153 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: Johannes Weberhofer (weberho) >Assigned to: Adrian Sandor (aditsu) Summary: encoding of last "-" character is not working correctly Initial Comment: in the test's case the last '-' character at the end of the paragraph is not encoded correctly. ---------------------------------------------------------------------- >Comment By: Adrian Sandor (aditsu) Date: 2010-08-03 01:05 Message: What version of jtidy did you use and what program/command did you run? ---------------------------------------------------------------------- Comment By: Johannes Weberhofer (weberho) Date: 2010-08-03 00:46 Message: I have attached jtidies' output. Have a look at the "incluindo – mas" section. I have tried to shorten the problematic sequence as short as I can. The strange thing is that - when you remove some things from the beginning, everything works nicely. ---------------------------------------------------------------------- Comment By: Adrian Sandor (aditsu) Date: 2010-08-02 23:10 Message: I opened both the input and output files in a browser and they look identical. Even the html source looks identical except for different line wrapping. What exactly is the problem? ---------------------------------------------------------------------- Comment By: Johannes Weberhofer (weberho) Date: 2010-08-02 22:58 Message: the '-' is a unicode character, not a regular - sign. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=113153&aid=3038314&group_id=13153 |
|
From: SourceForge.net <no...@so...> - 2010-08-02 16:46:51
|
Bugs item #3038314, was opened at 2010-08-02 16:50 Message generated for change (Comment added) made by weberho You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=113153&aid=3038314&group_id=13153 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: Johannes Weberhofer (weberho) Assigned to: Nobody/Anonymous (nobody) Summary: encoding of last "-" character is not working correctly Initial Comment: in the test's case the last '-' character at the end of the paragraph is not encoded correctly. ---------------------------------------------------------------------- >Comment By: Johannes Weberhofer (weberho) Date: 2010-08-02 18:46 Message: I have attached jtidies' output. Have a look at the "incluindo – mas" section. I have tried to shorten the problematic sequence as short as I can. The strange thing is that - when you remove some things from the beginning, everything works nicely. ---------------------------------------------------------------------- Comment By: Adrian Sandor (aditsu) Date: 2010-08-02 17:10 Message: I opened both the input and output files in a browser and they look identical. Even the html source looks identical except for different line wrapping. What exactly is the problem? ---------------------------------------------------------------------- Comment By: Johannes Weberhofer (weberho) Date: 2010-08-02 16:58 Message: the '-' is a unicode character, not a regular - sign. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=113153&aid=3038314&group_id=13153 |
|
From: SourceForge.net <no...@so...> - 2010-08-02 15:10:39
|
Bugs item #3038314, was opened at 2010-08-02 22:50 Message generated for change (Comment added) made by aditsu You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=113153&aid=3038314&group_id=13153 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: Johannes Weberhofer (weberho) Assigned to: Nobody/Anonymous (nobody) Summary: encoding of last "-" character is not working correctly Initial Comment: in the test's case the last '-' character at the end of the paragraph is not encoded correctly. ---------------------------------------------------------------------- >Comment By: Adrian Sandor (aditsu) Date: 2010-08-02 23:10 Message: I opened both the input and output files in a browser and they look identical. Even the html source looks identical except for different line wrapping. What exactly is the problem? ---------------------------------------------------------------------- Comment By: Johannes Weberhofer (weberho) Date: 2010-08-02 22:58 Message: the '-' is a unicode character, not a regular - sign. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=113153&aid=3038314&group_id=13153 |
|
From: SourceForge.net <no...@so...> - 2010-08-02 14:58:19
|
Bugs item #3038314, was opened at 2010-08-02 16:50 Message generated for change (Comment added) made by weberho You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=113153&aid=3038314&group_id=13153 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: Johannes Weberhofer (weberho) Assigned to: Nobody/Anonymous (nobody) Summary: encoding of last "-" character is not working correctly Initial Comment: in the test's case the last '-' character at the end of the paragraph is not encoded correctly. ---------------------------------------------------------------------- >Comment By: Johannes Weberhofer (weberho) Date: 2010-08-02 16:58 Message: the '-' is a unicode character, not a regular - sign. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=113153&aid=3038314&group_id=13153 |
|
From: SourceForge.net <no...@so...> - 2010-08-02 14:50:31
|
Bugs item #3038314, was opened at 2010-08-02 16:50 Message generated for change (Tracker Item Submitted) made by weberho You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=113153&aid=3038314&group_id=13153 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: Johannes Weberhofer (weberho) Assigned to: Nobody/Anonymous (nobody) Summary: encoding of last "-" character is not working correctly Initial Comment: in the test's case the last '-' character at the end of the paragraph is not encoded correctly. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=113153&aid=3038314&group_id=13153 |
31 messages has been excluded from this view by a project administrator.
