Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Commit 2d8cff4

Browse files
author
Chris Yang
authored
Refactor: fix typo "setup" -> "set up" (#43824)
There are several places in the engine using the word "setup" incorrectly. I have changed them to "set up" [C++, Objective-C, Java style guides]: https://github.com/flutter/engine/blob/main/CONTRIBUTING.md#style
1 parent 5d88e67 commit 2d8cff4

File tree

21 files changed

+98
-98
lines changed

21 files changed

+98
-98
lines changed

impeller/docs/read_frame_captures.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -211,7 +211,7 @@ function pipeline configuration.
211211

212212
![alt_text](assets/read_frame_captures/image7.png "image_tooltip")
213213

214-
You will get intimately familiar with this view when you setup a new pipeline
214+
You will get intimately familiar with this view when you set up a new pipeline
215215
state object in Impeller or try to reason about the correctness of one of the
216216
pipeline state object variants.
217217

impeller/docs/renderdoc_frame_capture.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22

33
[RenderDoc](https://renderdoc.org/) is a graphics debugger that can be used to capture frames. With Impeller starting to support gles and Vulkan backends, RenderDoc can provide insights into the application's frames.
44

5-
1. First step is to setup RenderDoc. Follow the instructions [here](https://renderdoc.org/docs/getting_started/quick_start.html). For the purposes of this guide it is assumed that you are able to get RenderDoc running.
5+
1. First step is to set up RenderDoc. Follow the instructions [here](https://renderdoc.org/docs/getting_started/quick_start.html). For the purposes of this guide it is assumed that you are able to get RenderDoc running.
66
2. The next step would be to run the application you wish the capture the frames of. Typically these would be one of the playground tests. To build these, do:
77

88
```bash

impeller/docs/xcode_frame_capture.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -89,7 +89,7 @@ that some of those diagnostics need Xcode to be able to re-compile the
8989
translation units in the engine. But, it doesn’t know how to do that, only
9090
GN/Ninja does. So some of those will be unavailable.
9191

92-
Any Impeller test that setup up a Playground will now automatically have GPU
92+
Any Impeller test that sets up a Playground will now automatically have GPU
9393
frame capture enabled.
9494

9595
# Select the Playground Enabled Test to Profile

impeller/playground/backend/vulkan/playground_impl_vk.cc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -108,7 +108,7 @@ PlaygroundImplVK::PlaygroundImplVK(PlaygroundSwitches switches)
108108

109109
vk::UniqueSurfaceKHR surface{vk_surface, context->GetInstance()};
110110
if (!context->SetWindowSurface(std::move(surface))) {
111-
VALIDATION_LOG << "Could not setup surface for context.";
111+
VALIDATION_LOG << "Could not set up surface for context.";
112112
return;
113113
}
114114

impeller/playground/playground.cc

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -125,8 +125,8 @@ void Playground::SetupContext(PlaygroundBackend backend) {
125125

126126
void Playground::SetupWindow() {
127127
if (!context_) {
128-
FML_LOG(WARNING)
129-
<< "Asked to setup a window with no context (call SetupContext first).";
128+
FML_LOG(WARNING) << "Asked to set up a window with no context (call "
129+
"SetupContext first).";
130130
return;
131131
}
132132
auto renderer = std::make_unique<Renderer>(context_);

impeller/renderer/backend/metal/context_mtl.mm

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -79,7 +79,7 @@ static bool DeviceSupportsComputeSubgroups(id<MTLDevice> device) {
7979
is_gpu_disabled_sync_switch_(std::move(is_gpu_disabled_sync_switch)) {
8080
// Validate device.
8181
if (!device_) {
82-
VALIDATION_LOG << "Could not setup valid Metal device.";
82+
VALIDATION_LOG << "Could not set up valid Metal device.";
8383
return;
8484
}
8585

@@ -135,7 +135,7 @@ static bool DeviceSupportsComputeSubgroups(id<MTLDevice> device) {
135135
resource_allocator_ = std::shared_ptr<AllocatorMTL>(
136136
new AllocatorMTL(device_, "Impeller Permanents Allocator"));
137137
if (!resource_allocator_) {
138-
VALIDATION_LOG << "Could not setup the resource allocator.";
138+
VALIDATION_LOG << "Could not set up the resource allocator.";
139139
return;
140140
}
141141
}
@@ -219,7 +219,7 @@ static bool DeviceSupportsComputeSubgroups(id<MTLDevice> device) {
219219
static id<MTLCommandQueue> CreateMetalCommandQueue(id<MTLDevice> device) {
220220
auto command_queue = device.newCommandQueue;
221221
if (!command_queue) {
222-
VALIDATION_LOG << "Could not setup the command queue.";
222+
VALIDATION_LOG << "Could not set up the command queue.";
223223
return nullptr;
224224
}
225225
command_queue.label = @"Impeller Command Queue";

impeller/renderer/backend/vulkan/context_vk.cc

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -227,7 +227,7 @@ void ContextVK::Setup(Settings settings) {
227227
std::make_unique<DebugReportVK>(*caps, device_holder->instance.get());
228228

229229
if (!debug_report->IsValid()) {
230-
VALIDATION_LOG << "Could not setup debug report.";
230+
VALIDATION_LOG << "Could not set up debug report.";
231231
return;
232232
}
233233

@@ -417,7 +417,7 @@ void ContextVK::Setup(Settings settings) {
417417

418418
//----------------------------------------------------------------------------
419419
/// Label all the relevant objects. This happens after setup so that the debug
420-
/// messengers have had a chance to be setup.
420+
/// messengers have had a chance to be set up.
421421
///
422422
SetDebugName(GetDevice(), device_holder_->device.get(), "ImpellerDevice");
423423
}

impeller/renderer/backend/vulkan/queue_vk.cc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ QueuesVK::QueuesVK(const vk::Device& device,
4343
auto vk_compute = device.getQueue(compute.family, compute.index);
4444
auto vk_transfer = device.getQueue(transfer.family, transfer.index);
4545

46-
// Always setup the graphics queue.
46+
// Always set up the graphics queue.
4747
graphics_queue = std::make_shared<QueueVK>(graphics, vk_graphics);
4848
ContextVK::SetDebugName(device, vk_graphics, "ImpellerGraphicsQ");
4949

impeller/renderer/command_buffer.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ class CommandBufferMock;
3434
///
3535
/// A command buffer is only meant to be used on a single thread. If
3636
/// a frame workload needs to be encoded from multiple threads,
37-
/// setup and record into multiple command buffers. The order of
37+
/// set up and record into multiple command buffers. The order of
3838
/// submission of commands encoded in multiple command buffers can
3939
/// be controlled via either the order in which the command buffers
4040
/// were created, or, using the `ReserveSpotInQueue` command which

lib/ui/painting/image_decoder_unittests.cc

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -406,12 +406,12 @@ TEST_F(ImageDecoderFixtureTest, ValidImageResultsInSuccess) {
406406
callback);
407407
};
408408

409-
auto setup_io_manager_and_decode = [&]() {
409+
auto set_up_io_manager_and_decode = [&]() {
410410
io_manager = std::make_unique<TestIOManager>(runners.GetIOTaskRunner());
411411
runners.GetUITaskRunner()->PostTask(decode_image);
412412
};
413413

414-
runners.GetIOTaskRunner()->PostTask(setup_io_manager_and_decode);
414+
runners.GetIOTaskRunner()->PostTask(set_up_io_manager_and_decode);
415415
latch.Wait();
416416
}
417417

@@ -706,12 +706,12 @@ TEST_F(ImageDecoderFixtureTest, ExifDataIsRespectedOnDecode) {
706706
callback);
707707
};
708708

709-
auto setup_io_manager_and_decode = [&]() {
709+
auto set_up_io_manager_and_decode = [&]() {
710710
io_manager = std::make_unique<TestIOManager>(runners.GetIOTaskRunner());
711711
runners.GetUITaskRunner()->PostTask(decode_image);
712712
};
713713

714-
runners.GetIOTaskRunner()->PostTask(setup_io_manager_and_decode);
714+
runners.GetIOTaskRunner()->PostTask(set_up_io_manager_and_decode);
715715

716716
latch.Wait();
717717

@@ -766,13 +766,13 @@ TEST_F(ImageDecoderFixtureTest, CanDecodeWithoutAGPUContext) {
766766
callback);
767767
};
768768

769-
auto setup_io_manager_and_decode = [&]() {
769+
auto set_up_io_manager_and_decode = [&]() {
770770
io_manager =
771771
std::make_unique<TestIOManager>(runners.GetIOTaskRunner(), false);
772772
runners.GetUITaskRunner()->PostTask(decode_image);
773773
};
774774

775-
runners.GetIOTaskRunner()->PostTask(setup_io_manager_and_decode);
775+
runners.GetIOTaskRunner()->PostTask(set_up_io_manager_and_decode);
776776

777777
latch.Wait();
778778
}

0 commit comments

Comments
 (0)