1.2.3
by Aron Xu
Import upstream version 3.3.2 |
1 |
CRUFT |
2 |
***** |
|
3 |
||
4 |
This file is designated for tracking cruft in our code. That is, code paths |
|
5 |
or ugly hacks that were put into place for a reason: Getting stuff to work |
|
6 |
but which could either been done better, or which time will obsolete. |
|
7 |
||
8 |
Examples are crude workarounds for broken compilers, libraries, OSes, or |
|
9 |
hardware, or or massive ``#ifdef`` clusters, that are better abstracted away |
|
10 |
into autoconf and convenience wrappers. |
|
11 |
||
12 |
If you are new to the project this is a good place to look for explanations |
|
13 |
of why things are the way they are, or help us fix things and make the code |
|
14 |
easier to read and maintain. |
|
15 |
||
16 |
||
17 |
Store.cc |
|
18 |
======== |
|
19 |
``iocore/cache/Store.cc`` contains three different ``#ifdef`` clusters with |
|
20 |
near identical code. Often you will read the same confused comment in three |
|
21 |
places, see for instance: http://issues.apache.org/jira/browse/TS-1707 |
|
22 |
||
1.1.11
by Arno Töll
Import upstream version 4.1.2 |
23 |
|
24 |
Java |
|
25 |
==== |
|
26 |
||
27 |
we should get rid of: ./example/protocol/test/*.java because, as zwoop says |
|
28 |
"friends don't let friends write code in Java". |
|
29 |
||
30 |
Plugins |
|
31 |
======= |
|
32 |
||
33 |
``geoip_acl`` should be a "helper plugin:, or a library/API which other |
|
34 |
plugins can use. |
|
35 |
||
36 |
||
37 |
Configuration & Defaults |
|
38 |
======================== |
|
39 |
||
40 |
Right now our server doesn't work without a reasonably filled records.config |
|
41 |
There are varying opinions on how this could or should be fixed, however one |
|
42 |
issue that arrises is that default configuration options are often set ad-hoc |
|
43 |
where ever they are needed as #defines. This should instead be consolidated |
|
44 |
into a single place. |