mirror of
https://github.com/mozilla/gecko-dev.git
synced 2024-11-25 05:41:12 +00:00
02a7b4ebdf
Allow-list all Python code in tree for use with the black linter, and re-format all code in-tree accordingly. To produce this patch I did all of the following: 1. Make changes to tools/lint/black.yml to remove include: stanza and update list of source extensions. 2. Run ./mach lint --linter black --fix 3. Make some ad-hoc manual updates to python/mozbuild/mozbuild/test/configure/test_configure.py -- it has some hard-coded line numbers that the reformat breaks. 4. Make some ad-hoc manual updates to `testing/marionette/client/setup.py`, `testing/marionette/harness/setup.py`, and `testing/firefox-ui/harness/setup.py`, which have hard-coded regexes that break after the reformat. 5. Add a set of exclusions to black.yml. These will be deleted in a follow-up bug (1672023). # ignore-this-changeset Differential Revision: https://phabricator.services.mozilla.com/D94045
40 lines
1.9 KiB
Python
40 lines
1.9 KiB
Python
# This Source Code Form is subject to the terms of the Mozilla Public
|
|
# License, v. 2.0. If a copy of the MPL was not distributed with this
|
|
# file, You can obtain one at http://mozilla.org/MPL/2.0/.
|
|
from __future__ import absolute_import
|
|
from __future__ import print_function
|
|
import os
|
|
from mozbuild.util import FileAvoidWrite
|
|
|
|
header_template = """#pragma GCC system_header
|
|
#pragma GCC visibility push(default)
|
|
{includes}
|
|
#pragma GCC visibility pop
|
|
"""
|
|
|
|
include_next_template = "#include_next <{header}>"
|
|
|
|
|
|
# The 'unused' arg is the output file from the file_generate action. We actually
|
|
# generate all the files in header_list
|
|
def gen_wrappers(unused, outdir, *header_list):
|
|
for header in header_list:
|
|
with FileAvoidWrite(os.path.join(outdir, header)) as f:
|
|
includes = include_next_template.format(header=header)
|
|
if header == "wayland-util.h":
|
|
# wayland-util.h in Wayland < 1.12 includes math.h inside an
|
|
# extern "C" block, which breaks including the header from C++.
|
|
# This was fixed in Wayland 1.12, but for versions earlier than
|
|
# that, we work around that by force-including math.h first.
|
|
includes = "#include <math.h>\n" + includes
|
|
elif header == "wayland-client.h":
|
|
# The system wayland-client.h uses quote includes for
|
|
# wayland-util.h, which means wayland-util.h is picked from the
|
|
# directory containing wayland-client.h first, and there's no
|
|
# way around that with -I, -isystem, or other flags. So, we
|
|
# force to include it from our wrapper, before including the
|
|
# system header, so that our wayland-util.h wrapper is picked
|
|
# first.
|
|
includes = '#include "wayland-util.h"\n' + includes
|
|
f.write(header_template.format(includes=includes))
|