mirror of
https://github.com/mozilla/gecko-dev.git
synced 2024-10-24 10:45:42 +00:00
b51e0fd0cc
> dom/media/gmp/CDMStorageIdProvider.cpp(63,10): warning:
> local variable 'storageId' will be copied despite being returned by name [-Wreturn-std-move]
nsAutoCString -> nsCString, will add std::move().
> layout/painting/DisplayItemClip.cpp(581,10): warning:
> local variable 'str' will be copied despite being returned by name [-Wreturn-std-move]
nsAutoCString -> nsCString, will add std::move().
> layout/painting/DisplayItemClipChain.cpp(88,10): warning:
> local variable 'str' will be copied despite being returned by name [-Wreturn-std-move]
nsAutoCString -> nsCString, will add std::move().
> layout/painting/nsDisplayList.cpp(179,10): warning:
> local variable 'str' will be copied despite being returned by name [-Wreturn-std-move]
nsAutoCString -> nsCString, will add std::move().
> gfx/thebes/gfxWindowsPlatform.cpp(454,10): warning:
> moving a local object in a return statement prevents copy elision [-Wpessimizing-move]
Will remove std::move().
> gfx/thebes/gfxFontEntry.cpp(245,20): warning:
> local variable 'name' will be copied despite being returned by name [-Wreturn-std-move]
nsAutoCString -> nsCString, will add std::move().
> netwerk/cookie/nsCookieService.cpp(4460,10): warning:
> local variable 'path' will be copied despite being returned by name [-Wreturn-std-move]
GetPathFromURI() result is stored in an nsAutoCString, so it might as well return that type.
> toolkit/components/extensions/WebExtensionPolicy.cpp(462,12): warning:
> local variable 'result' will be copied despite being returned by name [-Wreturn-std-move]
> toolkit/components/extensions/WebExtensionPolicy.cpp(475,10): warning:
> local variable 'result' will be copied despite being returned by name [-Wreturn-std-move]
`result` may be empty or may be arbitrarily long, so I'll use nsCString inside the function.
> toolkit/xre/CmdLineAndEnvUtils.h(349,10): warning:
> moving a local object in a return statement prevents copy elision [-Wpessimizing-move]
Returning an UniquePtr, will remove std::move().
Also will `return s` instead of `return nullptr` when `(!s)`, to avoid extra construction which could also prevent elision (not entirely sure, but it's at least not worse!); and it's clearer that the two `return`s return the same already-constructed on-stack object.
> tools/profiler/core/shared-libraries-win32.cc(111,10): warning:
> local variable 'version' will be copied despite being returned by name [-Wreturn-std-move]
nsPrintfCString -> nsCString, will add std::move().
> xpcom/glue/FileUtils.cpp(179,10): warning:
> local variable 'fullName' will be copied despite being returned by name [-Wreturn-std-move]
> xpcom/glue/FileUtils.cpp(209,10): warning:
> local variable 'path' will be copied despite being returned by name [-Wreturn-std-move]
nsAuto{,C}String -> ns{,C}String, will add std::move().
This allowed removals of 'AllowCompilerWarnings' from layout/painting,
netwerk/cookie, and toolkit/components/extensions.
Differential Revision: https://phabricator.services.mozilla.com/D5425
--HG--
extra : moz-landing-system : lando
78 lines
2.0 KiB
Python
78 lines
2.0 KiB
Python
# -*- Mode: python; indent-tabs-mode: nil; tab-width: 40 -*-
|
|
# vim: set filetype=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/.
|
|
|
|
with Files('**'):
|
|
BUG_COMPONENT = ('Core', 'Layout: Web Painting')
|
|
|
|
EXPORTS += [
|
|
'ActiveLayerTracker.h',
|
|
'DisplayItemClip.h',
|
|
'DisplayItemClipChain.h',
|
|
'DisplayListClipState.h',
|
|
'FrameLayerBuilder.h',
|
|
'LayerState.h',
|
|
'MatrixStack.h',
|
|
'nsCSSRenderingBorders.h',
|
|
'nsCSSRenderingGradients.h',
|
|
'nsDisplayItemTypes.h',
|
|
'nsDisplayItemTypesList.h',
|
|
'nsDisplayList.h',
|
|
'nsDisplayListInvalidation.h',
|
|
'nsImageRenderer.h',
|
|
'RetainedDisplayListBuilder.h',
|
|
'RetainedDisplayListHelpers.h',
|
|
'TransformClipNode.h',
|
|
]
|
|
|
|
EXPORTS.mozilla += [
|
|
'PaintTracker.h',
|
|
]
|
|
|
|
UNIFIED_SOURCES += [
|
|
'ActiveLayerTracker.cpp',
|
|
'DashedCornerFinder.cpp',
|
|
'DisplayItemClip.cpp',
|
|
'DisplayItemClipChain.cpp',
|
|
'DisplayListChecker.cpp',
|
|
'DisplayListClipState.cpp',
|
|
'DottedCornerFinder.cpp',
|
|
'FrameLayerBuilder.cpp',
|
|
'MaskLayerImageCache.cpp',
|
|
'nsCSSRendering.cpp',
|
|
'nsCSSRenderingBorders.cpp',
|
|
'nsCSSRenderingGradients.cpp',
|
|
'nsDisplayList.cpp',
|
|
'nsDisplayListInvalidation.cpp',
|
|
'nsImageRenderer.cpp',
|
|
'PaintTracker.cpp',
|
|
'RetainedDisplayListBuilder.cpp',
|
|
]
|
|
|
|
include('/ipc/chromium/chromium-config.mozbuild')
|
|
|
|
LOCAL_INCLUDES += [
|
|
'/docshell/base',
|
|
'/dom/base',
|
|
'/layout/base',
|
|
'/layout/generic',
|
|
'/layout/style',
|
|
'/layout/svg',
|
|
'/layout/tables',
|
|
'/layout/xul',
|
|
]
|
|
|
|
FINAL_LIBRARY = 'xul'
|
|
|
|
CXXFLAGS += CONFIG['MOZ_CAIRO_CFLAGS']
|
|
|
|
if CONFIG['CC_TYPE'] in ('msvc', 'clang-cl'):
|
|
# This is intended as a temporary hack to support building with VS2015.
|
|
# 'type cast': conversion from 'unsigned int' to 'void *' of greater size
|
|
CXXFLAGS += ['-wd4312']
|
|
|
|
if CONFIG['CC_TYPE'] in ('clang', 'gcc'):
|
|
CXXFLAGS += ['-Wno-error=shadow']
|