Checkout Tools
  • last updated 5 hours ago
Constraints: committers
Constraints: files
Constraints: dates
Fix various typos detected by codespell

* libsvn_client/add.c, libsvn_client/blame.c, libsvn_client/commit_util.c,

libsvn_client/conflicts.c, libsvn_client/copy.c, libsvn_client/diff.c,

libsvn_client/export.c, libsvn_client/import.c, libsvn_client/merge.c,

libsvn_client/mergeinfo.c, libsvn_client/mergeinfo.h, libsvn_client/patch.c,

libsvn_client/switch.c, libsvn_client/update.c, libsvn_delta/branch_compat.c,

libsvn_delta/compat.c, libsvn_delta/compose_delta.c, libsvn_delta/svndiff.c,

libsvn_delta/xdelta.c, libsvn_diff/deprecated.c, libsvn_diff/diff_file.c,

libsvn_fs/fs-loader.c, libsvn_fs_base/bdb/locks-table.c, libsvn_fs_base/fs.c,

libsvn_fs_base/key-gen.c, libsvn_fs_base/notes/fs-history,

libsvn_fs_base/reps-strings.c, libsvn_fs_base/tree.c,

libsvn_fs_fs/cached_data.c, libsvn_fs_fs/hotcopy.c, libsvn_fs_fs/index.c,

libsvn_fs_fs/low_level.c, libsvn_fs_fs/pack.c, libsvn_fs_fs/structure,

libsvn_fs_fs/temp_serializer.c, libsvn_fs_fs/transaction.c,

libsvn_fs_fs/tree.c, libsvn_fs_fs/util.h, libsvn_fs_x/cached_data.c,

libsvn_fs_x/caching.c, libsvn_fs_x/hotcopy.c, libsvn_fs_x/id.h,

libsvn_fs_x/index.c, libsvn_fs_x/low_level.c, libsvn_fs_x/pack.c,

libsvn_fs_x/rev_file.c, libsvn_fs_x/structure, libsvn_fs_x/temp_serializer.c,

libsvn_fs_x/transaction.c, libsvn_fs_x/tree.c, libsvn_fs_x/util.h,

libsvn_ra/ra_loader.c, libsvn_ra/ra_loader.h, libsvn_ra/util.c,

libsvn_ra/wrapper_template.h, libsvn_ra_local/ra_plugin.c,

libsvn_ra_serf/commit.c, libsvn_ra_serf/get_file.c,

libsvn_ra_serf/property.c, libsvn_ra_serf/ra_serf.h, libsvn_ra_serf/update.c,

libsvn_ra_serf/util.c, libsvn_ra_svn/client.c, libsvn_ra_svn/cyrus_auth.c,

libsvn_ra_svn/marshal.c, libsvn_repos/authz_parse.c, libsvn_repos/commit.c,

libsvn_repos/compat.c, libsvn_repos/hooks.c, libsvn_repos/load.c,

libsvn_repos/log.c, libsvn_repos/notify.c, libsvn_repos/repos.c,

libsvn_repos/repos.h, libsvn_subr/cache-memcache.c, libsvn_subr/cmdline.c,

libsvn_subr/config_file.c, libsvn_subr/date.c, libsvn_subr/gpg_agent.c,

libsvn_subr/io.c, libsvn_subr/object_pool.c, libsvn_subr/path.c,

libsvn_subr/prompt.c, libsvn_subr/string.c, libsvn_subr/sysinfo.c,

libsvn_subr/temp_serializer.c, libsvn_subr/utf.c, libsvn_subr/utf8proc.c,

libsvn_subr/version.c, libsvn_subr/win32_xlate.c, libsvn_subr/x509parse.c,

libsvn_wc/conflicts.c, libsvn_wc/conflicts.h, libsvn_wc/copy.c,

libsvn_wc/lock.c, libsvn_wc/wc_db.c, libsvn_wc/wc_db.h,


(): Spelling fixes

  1. … 97 more files in changeset.
BDB: Convert a silent propogation of corrupted keys into an assertion.

The callers don't bother to check for this condition. It shouldn't ever happen

and if it does something has gone terribly awry. Just run our malfunction

handler to avoid making a bigger mess.

* subversion/libsvn_fs_base/key-gen.c

(svn_fs_base__next_key): Just assert instead of returning an empty string.

Remove the overthought svn_fs_base__key_compare() function.

The old implmenetation ran strlen() on both strings, which in my opinion is a

really dubious optimization. I kinda doubt that walking both strings twice in

cases where the key is the same size is optimally better than saving the

character comparisons when they are different sizes. I'd bet that we're

comparing identically sized strings a lot of the time. Forcing -1, 0, or 1

as the only return values seems entirely pointless to me since the only thing

we ever do is test for 0 when calling this.

This incidentally fixes a compiler warning since we we were putting the return

of strlen into an int which isn't necessarily the same size as the strlen().

* subversion/libsvn_fs_base/key-gen.h

(svn_fs_base__key_compare): Convert the function to a macro to strcmp and

update the documentation to not promise things strcmp() doesn't promise.

* subversion/libsvn_fs_base/key-gen.c

(svn_fs_base__key_compare): Delete.

  1. … 1 more file in changeset.
Avoid potential integer overflow and underflow in svn_fs_base__next_key().

* subversion/libsvn_fs_base/key-gen.c

(svn_fs_base__next_key): Use the same type for the index as we received the

length as to avoid integer overflow. Avoid using a signed index since that

further reduces our range. Don't decrement until we're sure we can't

underflow. Rewrite the loop to avoid requiring a signed loop control

variable, using kfogel's favorite constant (BDB code is a blast from the


Test out my new and fancy ASF commit priviledges by changing the copyright

wording in our license headers to reflect ownership by the ASF.


Change terminology to ASF, and update a link.

* subversion/libsvn_subr/opt.c

(svn_opt__print_version_info): Note that the product as a whole is

copyrighted by the ASF, and update the project website.

* everywhere:

Change license text to reflect ASF ownership.

  1. … 891 more files in changeset.
Relicense Subversion under the Apache License, Version 2.0.

* NOTICE: New.



subversion/LICENSE: Remove.

* subversion/libsvn_subr/opt.c

(svn_opt__print_version_info): Note that the product as a whole is

copyrighted by the SVN Corp, and that it contains contributions from

many people, as referenced in NOTICE.

* subversion/bindings/swig/python/LICENSE_FOR_PYTHON_BINDINGS:

Relicense the SVN parts under Apache 2.0.

* everywhere:

Change copyright notices in file headers to reflect the Apache 2.0 license.

  1. … 882 more files in changeset.
Replace the expressions "? TRUE : FALSE" and "? FALSE : TRUE", which are ugly

and redundant or almost redundant, with more direct ways to express the

boolean test that is meant, throughout the C code base. No functional change.

The changes are similar to these:

boolean ? TRUE : FALSE => boolean

boolean ? FALSE : TRUE => ! (boolean)

x == y ? FALSE : TRUE => x != y

pointer ? TRUE : FALSE => pointer != NULL

integer ? FALSE : TRUE => integer != 0


  1. … 36 more files in changeset.
Move the skel code to libsvn_subr for private use by other libs.

- Renamed functions into svn_skel__* namespace

- Created #defines for old names, mapping to new. This keeps this change

to a small, manageable size.

- Adjusted #include lines for location of new header

- Applied const to some skel function params

- Switched some 0 values to NULL in the skel code (old -> new style)

* subversion/libsvn_fs_base/util/skel.h: renamed to ...

* subversion/include/private/svn_skel.h: here.

- renamed functions, add const to params

(svn_skel__getsize, svn_skel__putsize): moved from key-gen.h

* subversion/libsvn_fs_base/util/skel.c: renamed to ...

* subversion/libsvn_subr/skel.c: here.

- renamed functions, add const to params

(svn_skel__getsize, svn_skel__putsize): moved from key-gen.c

* subversion/libsvn_fs_base/key-gen.h:

(svn_fs_base__getsize, svn_fs_base__putsize): moved to svn_skel.h

* subversion/libsvn_fs_base/key-gen.c:

(svn_fs_base__getsize, svn_fs_base__putsize): moved to skel.c

* subversion/libsvn_fs_base/bdb/dbt.h:

* subversion/libsvn_fs_base/bdb/copies-table.c:

* subversion/libsvn_fs_base/bdb/lock-tokens-table.c:

* subversion/libsvn_fs_base/bdb/nodes-table.c:

* subversion/libsvn_fs_base/bdb/txn-table.c:

* subversion/libsvn_fs_base/bdb/locks-table.c:

* subversion/libsvn_fs_base/bdb/rev-table.c:

* subversion/libsvn_fs_base/util/fs_skels.h:

* subversion/libsvn_fs_base/util/fs_skels.c:

* subversion/tests/libsvn_fs_base/changes-test.c:

* subversion/tests/libsvn_fs_base/skel-test.c:

* subversion/tests/libsvn_fs_base/strings-reps-test.c:

- adjust includes

  1. … 17 more files in changeset.
Reformat the code to use a consistent no-space-before-param-list-paren style.

See here for the vote that decided this:

From: "Peter N. Lundblad" <>

To: "Peter N. Lundblad" <>


Subject: Results of: [VOTE] New space-before-parens style

Date: Wed, 8 Feb 2006 12:00:35 +0100 (CET)

Message-ID: <Pine.LNX.4.55.0602080918350.10805@localhost.localdomain>

I took the opportunity to make spacing among all parens in SVN_ERR

macro calls consistent. But I did not address consistency of casts,

nor whether a function name should always appear below its return type

in declarations and definitions, nor whether there should be a space

before/after the "*" when declaring pointer return types.

* www/hacking.html

(coding-style): Document that we are no-space-before-paren now.

* everywhere: Reformat.

  1. … 365 more files in changeset.
Merge the changes from the fs-abstraction branch. This allows the

libsvn_fs_fs filesystem to sit alongside the old BDB filesystem in the

same executable.

  1. … 89 more files in changeset.