Merge lp:~vila/bzr/dead-code into lp:bzr

Proposed by Vincent Ladeuil
Status: Merged
Approved by: John A Meinel
Approved revision: no longer in the source branch.
Merged at revision: 6550
Proposed branch: lp:~vila/bzr/dead-code
Merge into: lp:bzr
Diff against target: 28 lines (+0/-11)
1 file modified
bzrlib/filters/__init__.py (+0/-11)
To merge this branch: bzr merge lp:~vila/bzr/dead-code
Reviewer Review Type Date Requested Status
John A Meinel Approve
Review via email: mp+117874@code.launchpad.net

Commit message

Remove dead code bzrlib.filters.ContentFilterContext.config(): wasn't tested nor used.

Description of the change

This removes the config stuff in filters.ContentFilterContext (this was
untested (hence broken) never used and handle the case where a tree exists
without a branch which cannot be achieved in bzrlib).

To post a comment you must log in.
Revision history for this message
John A Meinel (jameinel) wrote :

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

On 8/2/2012 1:31 PM, Vincent Ladeuil wrote:
> Vincent Ladeuil has proposed merging lp:~vila/bzr/dead-code into
> lp:bzr.
>
> Requested reviews: bzr-core (bzr-core)
>
> For more details, see:
> https://code.launchpad.net/~vila/bzr/dead-code/+merge/117874
>
> This removes the config stuff in filters.ContentFilterContext (this
> was untested (hence broken) never used and handle the case where a
> tree exists without a branch which cannot be achieved in bzrlib).
>

 merge: approve

Based on your cover letter.

John
=:->

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.12 (Cygwin)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iEYEARECAAYFAlAad5YACgkQJdeBCYSNAAOKawCcCiwdOLQA1a4HCLU4bVrlYxGL
S4IAn0E/gcJWZvgalsgSf0F+t7NVDONO
=D3lV
-----END PGP SIGNATURE-----

review: Approve
Revision history for this message
Vincent Ladeuil (vila) wrote :

sent to pqm by email

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'bzrlib/filters/__init__.py'
2--- bzrlib/filters/__init__.py 2012-03-29 14:54:16 +0000
3+++ bzrlib/filters/__init__.py 2012-08-02 11:35:22 +0000
4@@ -90,7 +90,6 @@
5 # Cached values
6 self._revision_id = None
7 self._revision = None
8- self._config = None
9
10 def relpath(self):
11 """Relative path of file to tree-root."""
12@@ -131,16 +130,6 @@
13 self._revision = repo.get_revision(rev_id)
14 return self._revision
15
16- def config(self):
17- """The Config object to search for configuration settings."""
18- if self._config is None:
19- branch = getattr(self._tree, 'branch', None)
20- if branch is not None:
21- self._config = branch.get_config()
22- else:
23- self._config = config.GlobalConfig()
24- return self._config
25-
26
27 def filtered_input_file(f, filters):
28 """Get an input file that converts external to internal content.