mirror of
https://github.com/facebook/sapling.git
synced 2024-10-11 09:17:30 +03:00
918e1011b3
Summary:
This is a s//g replacement of all the `return (type_name) {` with
`return COMPOUND_LITERAL(type_name) {`.
This is the command which produced the diff:
`egrep 'return \(\w*\) \{' -Ir . --exclude='*.py*' --exclude-dir=.hg -l | xargs sed 's/return (\(\w*\)) {/return COMPOUND_LITERAL(\1) {/g' -i `
After I've done this, I checked:
`egrep '\(\w+\) \{' -Ir . --exclude='*.py*' --exclude-dir=.hg | egrep -v '(switch|while)' | grep -v 'if (' | grep -v 'COMPOUND_LITERAL' | less`
and it looks like the only things of `(something) {` syntax are function definitions, adding space before `(` in search pattern yields no results.
This is needed to make this compile on Windows under MSVC2015.
Depends on: D4843230
Test Plan:
- run `python setup.py build -f`, see it compile
- run all the tests, see them pass
Reviewers: #sourcecontrol, rmcelroy
Reviewed By: rmcelroy
Subscribers: rmcelroy, mjpieters
Differential Revision: https://phabricator.intern.facebook.com/D4843240
Signature: t1:4843240:1491495690:a097bfab403805052d5ac25d1db7bb32af3bca28
|
||
|---|---|---|
| .. | ||
| bitmanipulation.h | ||
| compat.h | ||
| datapackstore.cpp | ||
| datapackstore.h | ||
| key.h | ||
| match.h | ||
| mpatch.c | ||
| mpatch.h | ||
| py-cdatapack.h | ||
| py-cstore.cpp | ||
| py-datapackstore.h | ||
| py-structs.h | ||
| py-treemanifest.h | ||
| pythonutil.cpp | ||
| pythonutil.h | ||
| store.h | ||
| uniondatapackstore.cpp | ||
| uniondatapackstore.h | ||