| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
|
| |
Sometimes, they're there to emphasize a break between two sections of code,
which is good. In a majority of cases, though, they're just inconsistent.
|
| |
|
| |
|
|
|
|
|
|
|
|
| |
We don't actually use it. Two of contribs, pcre and sdl, reference it from
their autoconfage, so I'm not sure if it's optional or mandatory, and can't
check myself -- not deleting it yet. In any case, even a cursory look shows
at least two bugs, but I can't think of a case we could call this code,
even not directly.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
I had to rename distance() (in coord.h) to distance2() because it conflicts
with the STL function to compare 2 iterators. Not a bad change given how it
returns the square of the distance anyway.
I also had to rename the message global variable (in message.cc) to buffer.
I tried to fix and improve the coding style has much as I could, but I
probably missed a few given how huge and tedious it is.
I also didn't touch crawl-gdb.py, and the stuff in prebuilt, rltiles/tool
and util/levcomp.*, because I have no clue about those.
|
|
|
|
|
|
|
|
|
|
| |
This is a NIH implementation of a part of autoconf, it is limited as it
would be hard to affect the Makefile from such tests. On the other hand,
even nasty hacks like grepping for sqlite features are not enough here, as
we're looking for something in system headers.
I think all of configuration in the Makefile should be nuked and rewritten,
so it's just a crutch for 0.11.
|
|
|
|
|
|
|
| |
I used this function because some random page misled me into thinking it is
available on Windows (possibly a fault of my poor reading comprehension...).
It looks like that there's no existing function on Windows that's race-free,
so I implemented one from scratch.
|
| |
|
| |
|
|
|
|
|
|
| |
I'm not sure if the things I read are still true -- Sqlite folks analyzed
fsync() problems in OSX 10.3, they might have been fixed since; but it never
hurts to try the other way after the fcntl failed.
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
The same can happen on old systems written in days when disks did not have
internal caches yet, and on some disks which lie about flushes to score
higher in benchmarks. While the former is understandable and the latter has
been mostly eradicated due to outrage, there is no excuse for this behaviour
of fsync() when a working version exists.
So yeah, we need to use fcntl(fd, F_FULLFSYNC, 0) since this is how real
fsync() is named there.
|
|
|
|
|
|
| |
They're not that interesting and cloud code readability. What humans care
about is the conversion of Crawl's internal format to native strings (OUTS)
or native wide strings (OUTW, Windows only).
|
|\
| |
| |
| |
| | |
There are some issues left, like incorrect wrapping in some cases, but
we can fix them later.
|
| |\
| | |
| | |
| | | |
Conflicts galore...
|
| |\ \ |
|
| |\ \ \ |
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Only iostreams functions are left; on Windows they don't support Unicode
so a workaround will be needed.
|
| |_|_|/
|/| | |
| | | |
| | | |
| | | |
| | | |
| | | | |
"File:" is shown in your editor's status bar.
"Written by:" was used only for the first person who changed a file. We got
git for that now, and pre-DCSS history is so woefully inaccurate it doesn't
really matter.
|
| |_|/
|/| |
| | |
| | |
| | |
| | |
| | | |
Without it, starting Crawl twice in the same second produced identical seeds.
It doesn't matter for real games (no DGL on Windows...), but can be
surprising when doing automated tests or playing with layout generators.
|
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
It's strange that FreeBSD and old MacOS X don't have something that was
mandated in POSIX Issue 5 (1997), but let's cope with that. It's better
to unfairly accuse other systems of noncompliance and suffer a slowdown
than to fail to build at all.
|
| | | |
|
| | | |
|
| |/
|/| |
|
|/ |
|
| |
|
|
twice.
This fixes Mantis 2061.
I did not reuse the existing lk_open() because it's non-portable, waits for the
lock to become available and works using FILE* not a descriptor (bad for random
access).
|