Hi all!
I request your commentary on three changes I'd like to incorporate into
Freecell Solver:
1. At the moment, the Freecell Solver trunk, branches and tags Subversion
directries are directly at
http://svn.berlios.de/svnroot/repos/fc-solve/ .
However, there are now other projects (in their own mini-repositories) located
in the master fc-solve repository, which interfere with the history of FCS.
I'd like to move the trunk/branches/tags directories to their won mini-
repository. This will break all checkouts (which would be easily fixed using
an "svn switch" command), but should be more sustainable into the future.
2. Currently, the sizes of the packaged source archives of Freecell Solver
are:
{{{
210100 freecell-solver-2.26.0.tar.bz2
277588 freecell-solver-2.26.0.tar.gz
197044 freecell-solver-2.26.0.tar.xz
471007 freecell-solver-2.26.0.tar.Z
}}}
As one can see the tar.bz2 is much smaller than the tar.gz (and tar.xz which
so far few people know what it is, is even smaller). We've used tar.gz until
now, and I'm considering to switch to tar.bz2.
Does anyone object?
3. Lately, I've felt that the Freecell Solver sources have become
disorganised. This is especially evident in the headers:
{{{{
$ ls *.h
alloc.h cmd_line_enum.h fcs_enums.h fcs_user.h ms_ca.h state.h
app_str.h config.h fcs.h inline.h pqueue.h test_arr.h
caas.h fc_pro_iface_pos.h fcs_hash.h jhjtypes.h prefix.h tests.h
card.h fcs_cl.h fcs_isa.h lookup2.h preset.h
cl_chop.h fcs_dm.h fcs_move.h move.h rand.h
$
}}}}
It is not clear which of them are doing what according to their filenames, and
finding a function often requires using ctags or a search.
So I'm considering to re-organise them. Put all the sources under src/ and
make the filenames saner.
What do you think?
Regards,
Shlomi Fish
--
-----------------------------------------------------------------
Shlomi Fish http://www.shlomifish.org/
What does "Zionism" mean? - http://xrl.us/bjn8u
God gave us two eyes and ten fingers so we will type five times as much as we
read.
Received on Sun May 10 2009 - 21:33:18 IDT