- Jun 16, 2017
-
-
Matt Harbison authored
The error return is not 0 for this method, so _check() was doing nothing when an error occurred. This forces the error path, much like the check for OpenProcess(). The only unhandled return is now WAIT_ABANDONED, but I don't see how that could happen in this case.
-
- Jun 07, 2017
-
-
Matt Harbison authored
When I tried importing util.posixfile to work around removing a file opened by another process on Windows, it brought along the declarations in win32.py, which broke the error handling[1]. It doesn't seem worth hacking killdaemons[2] just to isolate these declarations in win32.py, so just declare them here to prevent any future issues. (win32.py mentions the declarations are required by pypy.) [1] https://www.mercurial-scm.org/pipermail/mercurial-devel/2017-May/097905.html [2] https://www.mercurial-scm.org/pipermail/mercurial-devel/2017-May/097907.html
-
- May 07, 2017
-
-
Matt Harbison authored
I've been using a local hghaveaddon.py to enable this for a couple of months with reasonable success, and 'killdaemons' is already enabled on Windows. There's one failure[1] in test-http-proxy.t that this adds, which I can't figure out. [1] https://www.mercurial-scm.org/pipermail/mercurial-devel/2017-April/096987.html
-
- Jun 15, 2017
-
-
Augie Fackler authored
This gives us an easy way to automatically update passing tests in Python 3.
-
Augie Fackler authored
-
Augie Fackler authored
-
Augie Fackler authored
-
Gregory Szorc authored
statprof.display_hotpath() accepts a "limit" function to choose the minimum threshold for samples to display. The default is 0.05, which means you don't need individual items contributing less than 5%. I had a need to adjust this threshold. We already have a config option for it. So let's reuse it. check-config.py doesn't like having multiple defaults for the ui.configwith() calls. The behavior is obviously correct. I'm not sure if it is worth teaching check-config.py how to ignore this. So I've just accepted the new output.
-
Gregory Szorc authored
-
Gregory Szorc authored
We previously weren't looking for this config helper. And, surprise, profiling.py references config options without docs. If I tried hard enough, I could have combined the regexps using a positive lookbehind assertion or something. But I didn't want to make my brain explode. At some point, we should probably do this linting at the tokenizer or ast layer. I'm not willing to open that can of worms right now.
-
Gregory Szorc authored
In preparation for making this regexp a bit more complicated.
-
Gregory Szorc authored
And split the regexp across multiple lines to make it easier to read.
-
Gregory Szorc authored
We didn't have explicit test coverage before. I broke check-config.py as part of writing patches and was lucky I realized it.
-
- Jun 16, 2017
-
-
Martin von Zweigbergk authored
The lazy locking is not used for bundle1, so using a regular context manager is clearer.
-
- Jun 15, 2017
-
-
Martin von Zweigbergk authored
-
- Jun 16, 2017
-
-
Martin von Zweigbergk authored
-
- Apr 11, 2017
-
-
Yuya Nishihara authored
-
- Apr 22, 2017
-
-
Yuya Nishihara authored
-
Yuya Nishihara authored
This will allow us to change the initial template reference depending on how the template is looked up. For example, -Tdefault => (ref='changeset', tmpl=None, mapfile='map-cmdline.default') -T'{rev}' => (ref='', tmpl='{rev}', mapfile=None) A literal template given by -T option will be stored as an unnamed template, which will free up the template namespace so that we can load named templates from [templates] section of user config.
-
Yuya Nishihara authored
A fewer number of arguments should be better.
-
Yuya Nishihara authored
I'm going to add more options to the templatespec tuple. cmdutil.logtemplatespec() is just an alias now, but it will be changed to a factory function later.
-
Yuya Nishihara authored
changeset_templater has lots of arguments, but most callers only need to specify a literal template 'tmpl'. "hg debugtemplate" has no diff option, which means 'opts' were effectively {}, so dropped opts.
-
- Jun 15, 2017
-
-
Matt Harbison authored
Without quoting, the output after "custom merge tool" in the next test is an abort trying to open "C:UsersMattAppData..."
-
- May 06, 2017
-
-
Yuya Nishihara authored
-
- Apr 22, 2017
-
-
Yuya Nishihara authored
This function is only useful when processing log options.
-
Yuya Nishihara authored
Since it's highly use-case dependent how template should be looked up, gettemplater() function isn't useful. Keeping it would introduce another bug I've made and fixed earlier in this series.
-
Yuya Nishihara authored
(tmpl, mapfile) will be packed into a named tuple later.
-
Yuya Nishihara authored
ui.graphnodetemplate config should be a literal template as ui.logtemplate is. The use of formatter.gettemplater() is valid only for a template string given by -T/--template option.
-
- May 06, 2017
-
-
Yuya Nishihara authored
This should have no effect, but seems good for code consistency.
-
Yuya Nishihara authored
I believe it was a mistake to open file in text mode. Now '\r' is preserved on Windows, but it should be okay to print '\r\n' as long as users live in such platform.
-
Yuya Nishihara authored
-
- Jun 13, 2017
-
-
Augie Fackler authored
Prior to this the return value was potentially None, a string, or a list of strings. It now always returns a list of strings where each string is always only one email address
-
Augie Fackler authored
Otherwise it's impossible to turn this feature back off, which is making writing of tests awkward.
-
Augie Fackler authored
-
- Jun 14, 2017
-
-
Brandon McCaig authored
Yuya Nishihara gave this suggestion on the mailing list after the previous patch was queued, and honestly this seems much simpler and looks more efficient.
-
- Jun 12, 2017
-
-
Martin von Zweigbergk authored
It was particularly unclear in phases.pushphase() whether the 0/1 returned were the 0/1 for public/draft phase or for False/True
-
- Apr 09, 2015
-
-
Yuya Nishihara authored
-1ms isn't significant, but seems not bad. revset #0: limit(0:9999, 100, 9000) 6) 0.001145 7) 0.000214 revset #3: last(0:9999, 100) 6) 0.000197 7) 0.000171
-
Yuya Nishihara authored
-1ms isn't significant, but seems not bad. revset #1: limit(0::9999, 100, 9000) 5) 0.001681 6) 0.000790
-
- Mar 23, 2015
-
-
Yuya Nishihara authored
Sub classes can provide optimized implementations.
-
- May 24, 2015
-
-
Yuya Nishihara authored
This renames the spanset class to _spanset, and moves its __init__ to new spanset() function. spanset() is now a factory function. This allows us to construct a spanset without keeping a repo instance.
-