[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
New Source tree (was: Upload of PenguinSound changes)
Peter Burns wrote:
>I moved the include files from PenguinPlay/Sound to PenguinPlay and
>removed the PenguinPlay/Sound directory.
Ok.
>I added very simple Thread and Mutex classes supporting both pthreads
>and windows threads.
Ok.
>I have removed the windows directory. The MSVC workspaces and projects
>are now organised as below.
>
>.\PenguinPlay.dsw
>.\src\PenguinPlay.dsp
>.\src\PenguinFile\PenguinFile.dsp
>.\src\PenguinSound\PenguinSound.dsp
>.\src\PenguinSound\psound_test.dsp
>
>PenguinFile.dsp does not build correctly as I am unsure of the libraries
>and preprocessor definitions that it uses.
No problem. The main projects/whatever file need to be adapted anyway, and
your copy of the PFile code lacks some mods by me. See below.
>When I built my module player using the new version of PenguinSound
>I get warnings about redefinitions of PACKAGE and VERSION as the
>configuration header for PenguinPlay is included from
>PenguinPlay/PenguinPlay.h and my module player also uses
>automake/autoconf with a configuration header.
Good point. What about adding a rule to include/PenguinPlay/Makefile.in :
ppconfig.h : config.h
sed -e "s/#define ([A-Z_])/#define PP$1/" \
-e "s/#undef ([A-Z_])/#undef PP$1/"
(not sure about exact sed regexp syntax)
and include only ppconfig.h?
>I also added an acconfig.h for the stuff that was commented out
>in configure.in and uncommented the commented stuff.
I left --enable-malloccheck and --enable-profile commented out, as they
need some more thinking (esp. --enable-malloccheck)
Ok, I added your files and modifications to my tree and adapted the
Makefile.ams (I also changed the lib_LIBRARIES in the PSound one to
noinst_LIBRARIES). I set the Lib's version to 0.0.2 for now.
Other changes for PFile:
* Completely removed ppfWin32POSIXImpl.*
* Renamed PakFile.* to PakArchive.*
* Renamed PakFileFile.* to PakFile.*
* Renamed PakFileDir.* to PakDirectory.*
* Renamed the other PakFile*.* files to Pak*.*
Changes for doc/PenguinFile:
* Removed BG-PakFile-Format1.sgml (we only have one format now)
* Moved the contents of BG-PakFile-Notes.sgml to BG-PakFile-Main.sgml
* Removed BG-PakFile-Notes.sgml
* Moved BG-PakFile*.sgml to BG-PakArchive*.sgml
Please note that I did *not* adapt the files' contents to that change yet!!!
Ok, in addition to that here's the changelog of my PFile modifications:
* Implemented/fixed the 64bit functions in PakFileUtils.*. They now all
use standard "long" data instead of pp(u)64 for portability. See the
comments in the header.
* Removed URLInfo::ToPlain () and URLInfo::IsForPlain ()
* Fixed bug in URLInfo::ToVirt (WIN32)
* Added GetVFSInterface () to DirEntry and properly implemented it for files
* Renamed ppFILE to ppfFILE
* Removed the DataSize / FileSize distinction for file interfaces.
* Switched the datasize / filesize meanings of the PakFileFile internals
back to the original meaning.
* Modified File and PakFile so that all Read/Write ops are now "piped"
through them. Makes the mechanism more extensible and distributes
responsibilities better. Not completely done yet.
* Modified the PakFile structure, dividing it into several "parts"
(header, extended header, file data, directory data)
* Moved DelPtr.h to the main include dir.
* Generalized internal API handlers from "XXXInPak" to "XXXVirt" (was
trivial)
* Fixed some bugs in ppfOpenDir () & ppfStat () (did not handle generic
"virt" URLs)
* Added PakFile * member to PakFileFile (still not handled properly though)
* little changes in /TODO and /AUTHORS
* some other minor things I forgot
!!!!!!!!!!!!!!!!!!!!!!!!
I am now committing this source tree to CVS
It will definitely not compile, but file names should be stable enough now.
PFile wouldn't even compile if I hadn't renamed/removed the files, as it's
a snapshot of my sources in the middle of some modifications.
!!!!!!!!!!!!!!!!!!!!!!!!
Christian
--
Drive A: not responding...Formatting C: instead