<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
<meta content="text/html;charset=UTF-8" http-equiv="Content-Type">
<title></title>
</head>
<body bgcolor="#ffffff" text="#000000">
Willie,<br>
I've checked in the code. It looks good. I cleaned up the
checkstyle complaints. Mostly removing whitespace at the end of the
lines and adding whitespace around operators. Also made changed "public
static int" to "public static final int" since they should never
change. Renamed BaseLFCustoms to AbstractLFCustoms. Moved member
variables to bottom of class. Added some javadoc. And I added the LGPL
license to each of the files with you as the author.<br>
The diffs would not apply. Either you can send the changed files or
try the patch again. If you are using Eclipse, you can go to
Synchronize view and for each package, right click on it and choose
"Create patch...". In the dialog, save it to disk. On the next screen,
check "Recurse into Subfolders" and "Include new files in patch" and
choose "Unified (format required by the Apply Patch)". Then click on
Finish. Because some mail readers wrap long lines even for attachments,
zip up the patches.<br>
<br>
Thanks so much for pitching in,<br>
DM<br>
<br>
Willie Thean wrote:
<blockquote cite="mid20050812012833.65474.qmail@web40724.mail.yahoo.com"
type="cite">
<pre wrap="">Hi DM --
Attached is the source and diff files.. Let me know if
it doesn't work.
I can send all the affected sources.
Thanks,
Willie
DM Smith wrote:
</pre>
<blockquote type="cite">
<pre wrap="">Willie,
It looks good to me! If you wish either sent a
</pre>
</blockquote>
<pre wrap=""><!---->unified patch
</pre>
<blockquote type="cite">
<pre wrap="">against the source or send the source and I'll
</pre>
</blockquote>
<pre wrap=""><!---->review and apply the
</pre>
<blockquote type="cite">
<pre wrap="">changes. Probably would be best to zip it.
Thanks!
DM
Willie Thean wrote:
</pre>
<blockquote type="cite">
<pre wrap="">Hi --
I clean the layout and UI a little for Metal and
WindowsXP LookAndFeel
(sorry I don't have a Mac). Put the attached
BDModified.zip in front of
the classpath and take a look.
Thanks,
Willie
____________________________________________________
Start your day with Yahoo! - make it your home page
<a class="moz-txt-link-freetext" href="http://www.yahoo.com/r/hs">http://www.yahoo.com/r/hs</a>
------------------------------------------------------------------------
_______________________________________________
jsword-devel mailing list
<a class="moz-txt-link-abbreviated" href="mailto:jsword-devel@crosswire.org">jsword-devel@crosswire.org</a>
<a class="moz-txt-link-freetext" href="http://www.crosswire.org/mailman/listinfo/jsword-devel">http://www.crosswire.org/mailman/listinfo/jsword-devel</a>
</pre>
</blockquote>
<pre wrap="">------------------------------------------------------------------------
_______________________________________________
jsword-devel mailing list
<a class="moz-txt-link-abbreviated" href="mailto:jsword-devel@crosswire.org">jsword-devel@crosswire.org</a>
<a class="moz-txt-link-freetext" href="http://www.crosswire.org/mailman/listinfo/jsword-devel">http://www.crosswire.org/mailman/listinfo/jsword-devel</a>
</pre>
</blockquote>
<pre wrap=""><!---->
____________________________________________________
Start your day with Yahoo! - make it your home page
<a class="moz-txt-link-freetext" href="http://www.yahoo.com/r/hs">http://www.yahoo.com/r/hs</a>
</pre>
<pre wrap="">
<hr size="4" width="90%">
_______________________________________________
jsword-devel mailing list
<a class="moz-txt-link-abbreviated" href="mailto:jsword-devel@crosswire.org">jsword-devel@crosswire.org</a>
<a class="moz-txt-link-freetext" href="http://www.crosswire.org/mailman/listinfo/jsword-devel">http://www.crosswire.org/mailman/listinfo/jsword-devel</a>
</pre>
</blockquote>
</body>
</html>