upstream/mercurial-mirror Files · contrib/vim/patchreview.txt

copies-rust: add smarter approach for merging small mapping with large mapping...

copies-rust: add smarter approach for merging small mapping with large mapping The current approach (finding the smaller updated set) works great when the mapping have similar size, but do a lot of unnecessary work when one side is tinier than the other one. So we do better in theses cases. See inline documentation for details. It give a sizeable boost to many of out slower cases: Repo Case Source-Rev Dest-Rev # of revisions old time new time Difference Factor time per rev --------------------------------------------------------------------------------------------------------------------------------------------------------------- mozilla-try x00000_revs_x_added_0_copies 1ebb79acd503 : 363753 revs, 18.123103 s, 5.693818 s, -12.429285 s, × 0.3142, 15 µs/rev mozilla-try x00000_revs_x_added_x_copies 95d83ee7242d : 362229 revs, 17.907312 s, 5.677655 s, -12.229657 s, × 0.3171, 15 µs/rev mozilla-try x00000_revs_x000_added_x_copies ca82787bb23c : 359344 revs, 17.684797 s, 5.563370 s, -12.121427 s, × 0.3146, 15 µs/rev mozilla-try x00000_revs_x0000_added_x0000_copies eb884023b810 : 192665 revs, 2.881471 s, 2.864099 s, -0.017372 s, × 0.9940, 14 µs/rev mozilla-try x00000_revs_x00000_added_x000_copies 8e29777b48e6 : 382065 revs, 63.148971 s, 59.498652 s, -3.650319 s, × 0.9422, 155 µs/rev mozilla-try x00000_revs_x00000_added_x000_copies 8e29777b48e6 : 382065 revs, 63.148971 s, 59.498652 s, -3.650319 s, × 0.9422, 155 µs/rev ideally, the im-rs object would have a `merge` method, but it does not (yet) Full timing comparison below (they are one pathological case than become even worse, for unclear reason). Repo Case Source-Rev Dest-Rev # of revisions old time new time Difference Factor time per rev --------------------------------------------------------------------------------------------------------------------------------------------------------------- mercurial x_revs_x_added_0_copies 39cfcef4f463 : 1 revs, 0.000043 s, 0.000042 s, -0.000001 s, × 0.9767, 42 µs/rev mercurial x_revs_x_added_x_copies 0c1d10351869 : 6 revs, 0.000105 s, 0.000104 s, -0.000001 s, × 0.9905, 17 µs/rev mercurial x000_revs_x000_added_x_copies dd3267698d84 : 1032 revs, 0.004895 s, 0.004913 s, +0.000018 s, × 1.0037, 4 µs/rev pypy x_revs_x_added_0_copies 099ed31b181b : 9 revs, 0.000194 s, 0.000191 s, -0.000003 s, × 0.9845, 21 µs/rev pypy x_revs_x000_added_0_copies 359343b9ac0e : 1 revs, 0.000050 s, 0.000050 s, +0.000000 s, × 1.0000, 50 µs/rev pypy x_revs_x_added_x_copies 72e022663155 : 7 revs, 0.000115 s, 0.000112 s, -0.000003 s, × 0.9739, 16 µs/rev pypy x_revs_x00_added_x_copies ace7255d9a26 : 1 revs, 0.000289 s, 0.000288 s, -0.000001 s, × 0.9965, 288 µs/rev pypy x_revs_x000_added_x000_copies a83dc6a2d56f : 6 revs, 0.010513 s, 0.010411 s, -0.000102 s, × 0.9903, 1735 µs/rev pypy x000_revs_xx00_added_0_copies 2f22446ff07e : 4785 revs, 0.051474 s, 0.052852 s, +0.001378 s, × 1.0268, 11 µs/rev pypy x000_revs_x000_added_x_copies 2c68e87c3efe : 6780 revs, 0.088086 s, 0.092828 s, +0.004742 s, × 1.0538, 13 µs/rev pypy x000_revs_x000_added_x000_copies 7b3dda341c84 : 5441 revs, 0.062176 s, 0.063269 s, +0.001093 s, × 1.0176, 11 µs/rev pypy x0000_revs_x_added_0_copies c9cb1334cc78 : 43645 revs, 0.720950 s, 0.711975 s, -0.008975 s, × 0.9876, 16 µs/rev pypy x0000_revs_xx000_added_0_copies 4ffed77c095c : 2 revs, 0.012897 s, 0.012771 s, -0.000126 s, × 0.9902, 6385 µs/rev pypy x0000_revs_xx000_added_x000_copies d9fa043f30c0 : 11316 revs, 0.121524 s, 0.124505 s, +0.002981 s, × 1.0245, 11 µs/rev netbeans x_revs_x_added_0_copies a01e9239f9e7 : 2 revs, 0.000082 s, 0.000082 s, +0.000000 s, × 1.0000, 41 µs/rev netbeans x_revs_x000_added_0_copies 20eb231cc7d0 : 2 revs, 0.000109 s, 0.000111 s, +0.000002 s, × 1.0183, 55 µs/rev netbeans x_revs_x_added_x_copies 5a39d12eecf4 : 3 revs, 0.000175 s, 0.000171 s, -0.000004 s, × 0.9771, 57 µs/rev netbeans x_revs_x00_added_x_copies 9eec5e90c05f : 9 revs, 0.000719 s, 0.000708 s, -0.000011 s, × 0.9847, 78 µs/rev netbeans x000_revs_xx00_added_0_copies 51d4ae7f1290 : 1421 revs, 0.010426 s, 0.010608 s, +0.000182 s, × 1.0175, 7 µs/rev netbeans x000_revs_x000_added_x_copies 6081d72689dc : 1533 revs, 0.015712 s, 0.015635 s, -0.000077 s, × 0.9951, 10 µs/rev netbeans x000_revs_x000_added_x000_copies 411350406ec2 : 5750 revs, 0.077353 s, 0.072072 s, -0.005281 s, × 0.9317, 12 µs/rev netbeans x0000_revs_xx000_added_x000_copies 1aad62e59ddd : 66949 revs, 0.673930 s, 0.682732 s, +0.008802 s, × 1.0131, 10 µs/rev mozilla-central x_revs_x_added_0_copies 7015fcdd43a2 : 2 revs, 0.000089 s, 0.000090 s, +0.000001 s, × 1.0112, 45 µs/rev mozilla-central x_revs_x000_added_0_copies 40d0c5bed75d : 8 revs, 0.000212 s, 0.000210 s, -0.000002 s, × 0.9906, 26 µs/rev mozilla-central x_revs_x_added_x_copies 14207ffc2b2f : 9 revs, 0.000183 s, 0.000182 s, -0.000001 s, × 0.9945, 20 µs/rev mozilla-central x_revs_x00_added_x_copies 446a150332c3 : 7 revs, 0.000595 s, 0.000594 s, -0.000001 s, × 0.9983, 84 µs/rev mozilla-central x_revs_x000_added_x000_copies 0a5e72d1b479 : 3 revs, 0.003117 s, 0.003102 s, -0.000015 s, × 0.9952, 1034 µs/rev mozilla-central x_revs_x0000_added_x0000_copies c07a39dc4e80 : 6 revs, 0.060197 s, 0.060234 s, +0.000037 s, × 1.0006, 10039 µs/rev mozilla-central x000_revs_xx00_added_0_copies 04a55431795e : 1593 revs, 0.006379 s, 0.006300 s, -0.000079 s, × 0.9876, 3 µs/rev mozilla-central x000_revs_x000_added_x_copies 2d37b966abed : 41 revs, 0.005008 s, 0.004817 s, -0.000191 s, × 0.9619, 117 µs/rev mozilla-central x000_revs_x000_added_x000_copies 4407bd0c6330 : 7839 revs, 0.065123 s, 0.065451 s, +0.000328 s, × 1.0050, 8 µs/rev mozilla-central x0000_revs_xx000_added_0_copies 67118cc6dcad : 615 revs, 0.026404 s, 0.026282 s, -0.000122 s, × 0.9954, 42 µs/rev mozilla-central x0000_revs_xx000_added_x000_copies 96a38b690156 : 30263 revs, 0.203456 s, 0.206873 s, +0.003417 s, × 1.0168, 6 µs/rev mozilla-central x00000_revs_x0000_added_x0000_copies 4c222a1d9a00 : 153721 revs, 1.929809 s, 1.935918 s, +0.006109 s, × 1.0032, 12 µs/rev mozilla-central x00000_revs_x00000_added_x000_copies 1daa622bbe42 : 204976 revs, 2.825064 s, 2.827320 s, +0.002256 s, × 1.0008, 13 µs/rev mozilla-try x_revs_x_added_0_copies 9790f499805a : 2 revs, 0.000857 s, 0.000842 s, -0.000015 s, × 0.9825, 421 µs/rev mozilla-try x_revs_x000_added_0_copies 5bb8ce8c7450 : 2 revs, 0.000870 s, 0.000870 s, +0.000000 s, × 1.0000, 435 µs/rev mozilla-try x_revs_x_added_x_copies 936255a0384a : 4 revs, 0.000161 s, 0.000165 s, +0.000004 s, × 1.0248, 41 µs/rev mozilla-try x_revs_x00_added_x_copies 017afae788ec : 2 revs, 0.001147 s, 0.001145 s, -0.000002 s, × 0.9983, 572 µs/rev mozilla-try x_revs_x000_added_x000_copies 6f0ee96e21ad : 1 revs, 0.026640 s, 0.026500 s, -0.000140 s, × 0.9947, 26500 µs/rev mozilla-try x_revs_x0000_added_x0000_copies c07a39dc4e80 : 6 revs, 0.059849 s, 0.059407 s, -0.000442 s, × 0.9926, 9901 µs/rev mozilla-try x000_revs_xx00_added_0_copies 04a55431795e : 1593 revs, 0.006326 s, 0.006325 s, -0.000001 s, × 0.9998, 3 µs/rev mozilla-try x000_revs_x000_added_x_copies 2d37b966abed : 41 revs, 0.005188 s, 0.005171 s, -0.000017 s, × 0.9967, 126 µs/rev mozilla-try x000_revs_x000_added_x000_copies 4c65cbdabc1f : 6657 revs, 0.067633 s, 0.066837 s, -0.000796 s, × 0.9882, 10 µs/rev mozilla-try x0000_revs_x_added_0_copies a36a2a865d92 : 40314 revs, 0.306969 s, 0.314252 s, +0.007283 s, × 1.0237, 7 µs/rev mozilla-try x0000_revs_x_added_x_copies bcabf2a78927 : 38690 revs, 0.293370 s, 0.304160 s, +0.010790 s, × 1.0368, 7 µs/rev mozilla-try x0000_revs_xx000_added_x_copies 4d0f2c178e66 : 8598 revs, 0.087159 s, 0.089223 s, +0.002064 s, × 1.0237, 10 µs/rev mozilla-try x0000_revs_xx000_added_0_copies 67118cc6dcad : 615 revs, 0.027251 s, 0.026711 s, -0.000540 s, × 0.9802, 43 µs/rev mozilla-try x0000_revs_xx000_added_x000_copies 7ccb2fc7ccb5 : 97052 revs, 3.010011 s, 3.243010 s, +0.232999 s, × 1.0774, 33 µs/rev mozilla-try x0000_revs_x0000_added_x0000_copies e951f4ad123a : 52031 revs, 0.753434 s, 0.756500 s, +0.003066 s, × 1.0041, 14 µs/rev mozilla-try x00000_revs_x_added_0_copies 1ebb79acd503 : 363753 revs, 18.123103 s, 5.693818 s, -12.429285 s, × 0.3142, 15 µs/rev mozilla-try x00000_revs_x00000_added_0_copies d16fde900c9c : 34414 revs, 0.583206 s, 0.590904 s, +0.007698 s, × 1.0132, 17 µs/rev mozilla-try x00000_revs_x_added_x_copies 95d83ee7242d : 362229 revs, 17.907312 s, 5.677655 s, -12.229657 s, × 0.3171, 15 µs/rev mozilla-try x00000_revs_x000_added_x_copies ca82787bb23c : 359344 revs, 17.684797 s, 5.563370 s, -12.121427 s, × 0.3146, 15 µs/rev mozilla-try x00000_revs_x0000_added_x0000_copies eb884023b810 : 192665 revs, 2.881471 s, 2.864099 s, -0.017372 s, × 0.9940, 14 µs/rev mozilla-try x00000_revs_x00000_added_x0000_copies 1ae03d022d6d : 228985 revs, 101.062002 s, 113.297287 s, +12.235285 s, × 1.1211, 494 µs/rev mozilla-try x00000_revs_x00000_added_x000_copies 8e29777b48e6 : 382065 revs, 63.148971 s, 59.498652 s, -3.650319 s, × 0.9422, 155 µs/rev Differential Revision: https://phab.mercurial-scm.org/D9491

timeless@mozdev.org - - Load All Authors

File last commit:

r26098:ce26928c default


                r46744:c94d013e

default

Download file

             patchreview.txt
        
                    97 lines
            
             | 3.9 KiB
            
                | text/plain
            
             |
                TextLexer

/ contrib / vim / patchreview.txt

History | Source | Raw |Copy content |Copy permalink

Manpreet Singh Updated contrib/vim/patchreview.* to version 0.2.1...	r10545	patchreview.txt Vim global plugin for doing single, multi-patch or diff code reviews
Manpreet Singh Updated contrib/vim/patchreview.* to version 0.2.2...	r10550	Version v0.2.2 (for Vim version 7.0 or higher)
Manpreet Singh contrib: patch review plugin for vim 7.0...	r2350
Manpreet Singh Updated contrib/vim/patchreview.* to version 0.2.1...	r10545	Author: Manpreet Singh < junkblocker@yahoo.com >
		Copyright (C) 2006-2010 by Manpreet Singh
Manpreet Singh contrib: patch review plugin for vim 7.0...	r2350	License : This file is placed in the public domain.

		=============================================================================

Manpreet Singh Updated contrib/vim/patchreview.* to version 0.2.1...	r10545	CONTENTS patchreview diffreview patchreview-contents
Manpreet Singh contrib: patch review plugin for vim 7.0...	r2350
		1. Contents.........................................: \|patchreview-contents\|
		2. Introduction.....................................: \|patchreview-intro\|
		3. PatchReview options..............................: \|patchreview-options\|
		4. PatchReview Usage................................: \|patchreview-usage\|
Manpreet Singh Updated contrib/vim/patchreview.* to version 0.2.1...	r10545	4.1 DiffReview Usage.............................: \|:DiffReview\|
		4.2 PatchReview Usage............................: \|:PatchReview\|
Manpreet Singh contrib: patch review plugin for vim 7.0...	r2350
		=============================================================================

		PatchReview Introduction patchreview-intro

Manpreet Singh Updated contrib/vim/patchreview.* to version 0.2.1...	r10545	The Patch Review plugin allows easy single or multipatch code or diff reviews.

		It opens each affected file in the patch or in a workspace diff in a diff view
		in a separate tab.

		VIM provides the \|:diffpatch\| and related commands to do single file reviews
		but can not handle patch files containing multiple patches as is common with
		software development projects. This plugin provides that missing
		functionality.

timeless@mozdev.org spelling: behaviour -> behavior	r26098	It also improves on \|:diffpatch\|'s behavior of creating the patched files in
Manpreet Singh contrib: patch review plugin for vim 7.0...	r2350	the same directory as original file which can lead to project workspace
		pollution.

Manpreet Singh Updated contrib/vim/patchreview.* to version 0.2.2...	r10550	It does automatic diff generation for various version control systems by
		running their diff command.

Manpreet Singh contrib: patch review plugin for vim 7.0...	r2350	=============================================================================

		PatchReview Options patchreview-options

Manpreet Singh Updated contrib/vim/patchreview.* to version 0.2.2...	r10550	g:patchreview_patch = {string}
		Optional path to patch binary. PatchReview tries to locate patch on
		system path automatically. If the binary is not on system path, this
		option tell PatchReview the full path to the binary. This option, if
		specified, overrides the default patch binary on the path.
Manpreet Singh Updated contrib/vim/patchreview.* to version 0.2.1...	r10545
		examples:
Manpreet Singh Updated contrib/vim/patchreview.* to version 0.2.2...	r10550	(On Windows with Cygwin) >
		let g:patchreview_patch = 'c:\\cygwin\\bin\\patch.exe'
Manpreet Singh Updated contrib/vim/patchreview.* to version 0.2.1...	r10545	<
		(On *nix systems) >
Manpreet Singh Updated contrib/vim/patchreview.* to version 0.2.2...	r10550	let g:patchreview_patch = '/usr/bin/gpatch'
Manpreet Singh Updated contrib/vim/patchreview.* to version 0.2.1...	r10545	<

		g:patchreview_filterdiff = {string}
		Optional path to filterdiff binary. PatchReview tries to locate
		filterdiff on system path automatically. If the binary is not on system
		path, this option tell PatchReview the full path to the binary. This
		option, if specified, overrides the default filterdiff binary on the
		path.
Manpreet Singh contrib: patch review plugin for vim 7.0...	r2350
		examples:
		(On Windows with Cygwin)
Manpreet Singh Updated contrib/vim/patchreview.* to version 0.2.1...	r10545	>
Manpreet Singh contrib: patch review plugin for vim 7.0...	r2350	let g:patchreview_filterdiff = 'c:\\cygwin\\bin\\filterdiff.exe'
Manpreet Singh Updated contrib/vim/patchreview.* to version 0.2.1...	r10545	<
Manpreet Singh contrib: patch review plugin for vim 7.0...	r2350	(On *nix systems)
Manpreet Singh Updated contrib/vim/patchreview.* to version 0.2.1...	r10545	>
Manpreet Singh contrib: patch review plugin for vim 7.0...	r2350	let g:patchreview_filterdiff = '/usr/bin/filterdiff'
Manpreet Singh Updated contrib/vim/patchreview.* to version 0.2.1...	r10545	<
Manpreet Singh contrib: patch review plugin for vim 7.0...	r2350	=============================================================================

		PatchReview Usage patchreview-usage
Manpreet Singh Updated contrib/vim/patchreview.* to version 0.2.1...	r10545	:DiffReview

		:DiffReview

		Perform a diff review in the current directory under version control.
		Currently supports Mercurial (hg), Subversion (svn), CVS, Bazaar (bzr) and
		Monotone.

Manpreet Singh contrib: patch review plugin for vim 7.0...	r2350	:PatchReview

		:PatchReview patchfile_path [optional_source_directory]

		Perform a patch review in the current directory based on the supplied
		patchfile_path. If optional_source_directory is specified, patchreview is
Manpreet Singh Updated contrib/vim/patchreview.* to version 0.2.1...	r10545	done on that directory. Otherwise, the current directory is assumed to be
Manpreet Singh contrib: patch review plugin for vim 7.0...	r2350	the source directory.
Manpreet Singh Updated contrib/vim/patchreview.* to version 0.2.1...	r10545
		Only supports context or unified format patches.

		------------------------------------------------------------------------------
		vim: ft=help:ts=2:sts=2:sw=2:tw=78:norl:

	Site-wide shortcuts
/	Use quick search box
g h	Goto home page
g g	Goto my private gists page
g G	Goto my public gists page
g 0-9	Goto bookmarked items from 0-9
n r	New repository page
n g	New gist page

	Repositories
g s	Goto summary page
g c	Goto changelog page
g f	Goto files page
g F	Goto files page with file search activated
g p	Goto pull requests page
g o	Goto repository settings
g O	Goto repository access permissions settings
t s	Toggle sidebar on some pages