-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On Sun, 7 Jun 2009, Johannes Berg wrote:

Hallo Johannes,

Nice. I gave it an almost complete rewrite again, getting rid of the
magic array indexes and things like that. Please test, I haven't had a

es gab einen off-by-one Bug beim Laden der Config. Dort habe ich noch eine "2" durch den Enum-Wert ersetzt.

Beim Test sieht's super aus!

Man koennte jetzt noch match_fn[] in die match_name[] packen, um alles zusammen zu haben ... .

Wie ist das eigentlich, das Feld, auf den der Rueckgabewert von getenv()
zeigt, darf nicht veraendert werden, oder? Ansonsten koennte man:

                        strings[i] = p_strsplit(global_pool, tmp, ";");
                        if (i == MT_PATTERN_IGNCASE) {
                                /* lower case the string */
                                char **list = strings[i];
                                while (*list) {
                                        lowercase_string(*list, *list);
                                        ++list;
                                }
                        }

durch

                        if (i == MT_PATTERN_IGNCASE) {
                                lowercase_string(tmp, tmp);
                        }
                        strings[i] = p_strsplit(global_pool, tmp, ";");

ersetzen und Zeit sparen.

Tschuess,

- -- Steffen Kaiser
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)

iQEVAwUBSi+sAHWSIuGy1ktrAQJVZgf+JpKXXSh2kwthE47OmsGLriaDhRsbiIkx
6ikOgxbvpsHXKri2h9Y0+NSG5mQnfOvT6AAz1dtapJvXxj+Z84s35xzu3xCfjnNV
ent383Go6/7lrqJDvwU6Gvg3yQmXGLDPwI7OKTOK6nrcIOwrOfEx/KBfIqfEBMAp
P/uxzFpUgwbDduWakKb0YobciXKm/WQ4FzP/gvHVr5cMMXKUi8Cn+EPhqE6U9gNI
jwriJyxWNLC+/YIzJZZtmaRb8QXala8gGdIa56mNu2iR9p4JeYIOBWY0UgGWqKV+
TtPRm9g+DZUi3wd5avnt7YRkAbiwVw9T3mrZ9TmDHW05ZCTg41MAKA==
=kxfl
-----END PGP SIGNATURE-----

Attachment: antispam.off-by-one.patch.bz2
Description: Binary data

Reply via email to