Re: I'd like to be a developer for HTML Tidy

>> I suggest you create SourceForge Tracker items (Patches)
>> and attach your patches to those items, and/or describe the
>> following patches in more detail, to the mailing list. The
>> project admins usually like to see how serious interested
>> developers are, and the quality of their work, before giving
>> them CVS commit privileges.

That makes loads of sense. I was wondering about how you do code review, and
I was concerned about testing on other platforms,  The Patches approach is
exactly what I was looking for.

I've submitted my first batch of changes as [ 1239422 ] Cleanup: explicit
casts, native booleans (C++), minor fix

These changes use explicit casts for previously implicit type conversions,
use an optional #define to support intrinsic booleans (as in C++), and fixes
two possible bugs where the return values of tidySetOptionCallback() and
tidySetReportFilter() might be unpredictable.

Eric


-----Original Message-----
From: Terry Teague [mailto:[email protected]]
Sent: July 15, 2005 1:22 PM
To: Eric Hartwell
Cc: [email protected]
Subject: Re: I'd like to be a developer for HTML Tidy

At 6:10 AM -0700 7/14/05, Eric Hartwell wrote:
>Message body follows:
>
>attn: Charles Reitzel, Björn Höhrmann, Terry Teague


I have forwarded this reply to the Tidy-develop mailing list. I would
recommend you sign up for this mailing list if you haven't already done so.


>I'd like to contribute to the HTML Tidy project by becoming a 
>developer. I have loads of time right now while I'm looking for a
>(paying) job.


Cool, we are always looking for people who have the time to contribute to
the project.


>I've been experimenting with HTML Tidy in .NET and I'd like to 
>contribue my work back to the project.by becoming a developer.


I'm not a .NET user, but there are a number of other developers who have
.NET experience.


>There are already a number of SourceForge projects to build COM/.NET 
>Tidy wrappers, but they all use an old code base.
>The best way to preserve and extend the core Tidy functionality is to 
>create TidyLib API wrappers as sub- projects of the core project and 
>propagate fixes back into the core code.


I suggest you create SourceForge Tracker items (Patches) and attach your
patches to those items, and/or describe the following patches in more
detail, to the mailing list. The project admins usually like to see how
serious interested developers are, and the quality of their work, before
giving them CVS commit privileges.


>Core code cleanup:
>1. Add explicit casts where implicit casts are used for (e.g.
>enum <=> uint, size_t <=> int)
>2. Add a NATIVE_BOOL (or similar) preprocessor definition to use 
>compiler's bool, true, false when available 3. A few preemptive bug 
>fixes (e.g. #1004512 fixed in one place but not another).
>
>Core code extension:
>1. Add stream interface for config.c option file processing (so you can 
>specify options without using a disk file)
>
>TidyATL++
>1. Thin COM wrapper providing Unicode load/save from memory/file, load 
>config from memory, cleanup, diagnostics 2. Unicode <==> UTF-16, BSTR 
><==> tmbstr 3. The current code base seems to work when compiled with 
>Microsoft C++ instead of C, but I don't see any particular advantage 
>other than the enhanced compiler warnings.
>
>Other projects:
>
>.NET
>The current TidyLib code compiles as C++, but it cannot run in .NET 
>native mode. This is mostly because of the use of variable arguments in 
>the formatting/error reporting functions.
>It's possible to refactor the C code to avoid this (cutting the lines 
>of code significantly), but I'm not sure if it's worth doing, unless 
>the community decides to refactor the entire project.
>
>Tidy.NET
>crocgod's Tidy.NET is a native .NET port to C#. It compiles and runs in 
>.NET 2.0 beta 2. The code is apparently based on a 2002 or 2003 version 
>of TidyLib. It exposes TidyLib's document/node interface in W3C DOM 
>format, but I'm not sure how complete or useful that is. The 
>configuration options are exposed as individual named variables.
>
>Tidy.NET++
>1. I've written a code generator that reads the current config.c file 
>and generates C# code for an Options class that that provides a cleaner 
>configuration interface (Bool options exposed as enums, named option 
>get/set, etc.).


Regards, Terry




-------------------------------------------------------
SF.Net email is sponsored by: Discover Easy Linux Migration Strategies
from IBM. Find simple to follow Roadmaps, straightforward articles,
informative Webcasts and more! Get everything you need to get up to
speed, fast. http://ads.osdn.com/?ad_idt77&alloc_id492&op=click