mirror of
https://github.com/RGBCube/serenity
synced 2025-07-26 23:37:36 +00:00
Userland: Use Core::Timer::create_foo() factory functions where possible
This commit is contained in:
parent
6edc0cf5ab
commit
e181b1cb82
12 changed files with 37 additions and 51 deletions
|
@ -31,8 +31,7 @@
|
|||
#include <unistd.h>
|
||||
|
||||
HexEditor::HexEditor()
|
||||
: m_blink_timer(Core::Timer::construct())
|
||||
, m_document(make<HexDocumentMemory>(ByteBuffer::create_zeroed(0).release_value_but_fixme_should_propagate_errors()))
|
||||
: m_document(make<HexDocumentMemory>(ByteBuffer::create_zeroed(0).release_value_but_fixme_should_propagate_errors()))
|
||||
{
|
||||
set_should_hide_unnecessary_scrollbars(true);
|
||||
set_focus_policy(GUI::FocusPolicy::StrongFocus);
|
||||
|
@ -42,11 +41,10 @@ HexEditor::HexEditor()
|
|||
set_foreground_role(ColorRole::BaseText);
|
||||
vertical_scrollbar().set_step(line_height());
|
||||
|
||||
m_blink_timer->set_interval(500);
|
||||
m_blink_timer->on_timeout = [this]() {
|
||||
m_blink_timer = Core::Timer::create_repeating(500, [this]() {
|
||||
m_cursor_blink_active = !m_cursor_blink_active;
|
||||
update();
|
||||
};
|
||||
}).release_value_but_fixme_should_propagate_errors();
|
||||
m_blink_timer->start();
|
||||
}
|
||||
|
||||
|
|
|
@ -85,7 +85,7 @@ private:
|
|||
size_t m_position { 0 };
|
||||
bool m_cursor_at_low_nibble { false };
|
||||
EditMode m_edit_mode { Hex };
|
||||
NonnullRefPtr<Core::Timer> m_blink_timer;
|
||||
RefPtr<Core::Timer> m_blink_timer;
|
||||
bool m_cursor_blink_active { false };
|
||||
NonnullOwnPtr<HexDocument> m_document;
|
||||
GUI::UndoStack m_undo_stack;
|
||||
|
|
|
@ -47,10 +47,10 @@ ErrorOr<int> serenity_main(Main::Arguments arguments)
|
|||
window->resize(840, 600);
|
||||
window->set_icon(app_icon.bitmap_for_size(16));
|
||||
|
||||
auto main_widget_updater = Core::Timer::construct(static_cast<int>((1 / 30.0) * 1000), [&] {
|
||||
auto main_widget_updater = TRY(Core::Timer::create_repeating(static_cast<int>((1 / 30.0) * 1000), [&] {
|
||||
if (window->is_active())
|
||||
Core::EventLoop::current().post_event(main_widget, make<Core::CustomEvent>(0));
|
||||
});
|
||||
}));
|
||||
main_widget_updater->start();
|
||||
|
||||
auto& file_menu = window->add_menu("&File");
|
||||
|
|
|
@ -22,11 +22,9 @@ namespace PixelPaint {
|
|||
|
||||
SprayTool::SprayTool()
|
||||
{
|
||||
m_timer = Core::Timer::construct();
|
||||
m_timer->on_timeout = [&]() {
|
||||
m_timer = Core::Timer::create_repeating(200, [&]() {
|
||||
paint_it();
|
||||
};
|
||||
m_timer->set_interval(200);
|
||||
}).release_value_but_fixme_should_propagate_errors();
|
||||
}
|
||||
|
||||
static double nrand()
|
||||
|
|
|
@ -36,11 +36,9 @@ TextTool::TextTool()
|
|||
m_text_editor->set_wrapping_mode(GUI::TextEditor::WrappingMode::NoWrap);
|
||||
m_selected_font = Gfx::FontDatabase::default_font();
|
||||
m_text_editor->set_font(m_selected_font);
|
||||
m_cursor_blink_timer = Core::Timer::construct();
|
||||
m_cursor_blink_timer->on_timeout = [&]() {
|
||||
m_cursor_blink_timer = Core::Timer::create_repeating(500, [&]() {
|
||||
m_cursor_blink_state = !m_cursor_blink_state;
|
||||
};
|
||||
m_cursor_blink_timer->set_interval(500);
|
||||
}).release_value_but_fixme_should_propagate_errors();
|
||||
}
|
||||
|
||||
void TextTool::on_tool_deactivation()
|
||||
|
|
|
@ -11,11 +11,11 @@ PlaybackManager::PlaybackManager(NonnullRefPtr<Audio::ConnectionToServer> connec
|
|||
: m_connection(connection)
|
||||
{
|
||||
// FIXME: The buffer enqueuing should happen on a wholly independent second thread.
|
||||
m_timer = Core::Timer::construct(PlaybackManager::update_rate_ms, [&]() {
|
||||
m_timer = Core::Timer::create_repeating(PlaybackManager::update_rate_ms, [&]() {
|
||||
if (!m_loader)
|
||||
return;
|
||||
next_buffer();
|
||||
});
|
||||
}).release_value_but_fixme_should_propagate_errors();
|
||||
m_device_sample_rate = connection->get_sample_rate();
|
||||
}
|
||||
|
||||
|
|
|
@ -66,17 +66,14 @@ ErrorOr<int> serenity_main(Main::Arguments arguments)
|
|||
(void)TRY(advice_widget->try_set_layout<GUI::VerticalBoxLayout>());
|
||||
advice_widget->layout()->set_spacing(0);
|
||||
|
||||
auto advice_timer = TRY(Core::Timer::try_create());
|
||||
advice_timer->set_interval(15'000);
|
||||
advice_timer->set_single_shot(true);
|
||||
advice_timer->on_timeout = [&] {
|
||||
auto advice_timer = TRY(Core::Timer::create_single_shot(15'000, [&] {
|
||||
window->move_to_front();
|
||||
advice_window->move_to_front();
|
||||
catdog_widget->set_roaming(false);
|
||||
advice_window->move_to(window->x() - advice_window->width() / 2, window->y() - advice_window->height());
|
||||
advice_window->show();
|
||||
advice_window->set_always_on_top();
|
||||
};
|
||||
}));
|
||||
advice_timer->start();
|
||||
|
||||
advice_widget->on_dismiss = [&] {
|
||||
|
|
|
@ -41,26 +41,28 @@ DemoWizardDialog::DemoWizardDialog(GUI::Window* parent_window)
|
|||
.release_value_but_fixme_should_propagate_errors();
|
||||
m_page_2->body_widget().load_from_gml(demo_wizard_page_2_gml).release_value_but_fixme_should_propagate_errors();
|
||||
m_page_2_progressbar = m_page_2->body_widget().find_descendant_of_type_named<GUI::Progressbar>("page_2_progressbar");
|
||||
m_page_2_timer = Core::Timer::try_create(this).release_value_but_fixme_should_propagate_errors();
|
||||
m_page_2_timer = Core::Timer::create_repeating(
|
||||
100, [&]() {
|
||||
if (m_page_2_progress_value < 100)
|
||||
m_page_2_progress_value++;
|
||||
m_page_2_progressbar->set_value(m_page_2_progress_value);
|
||||
|
||||
// Go to final page on progress completion
|
||||
if (m_page_2_progress_value == 100) {
|
||||
m_page_2_progress_value = 0;
|
||||
replace_page(*m_back_page);
|
||||
}
|
||||
},
|
||||
this)
|
||||
.release_value_but_fixme_should_propagate_errors();
|
||||
m_page_2->on_page_enter = [&]() {
|
||||
m_page_2_progress_value = 0;
|
||||
m_page_2_timer->restart(100);
|
||||
m_page_2_timer->restart();
|
||||
};
|
||||
m_page_2->on_page_leave = [&]() {
|
||||
m_page_2_progress_value = 0;
|
||||
m_page_2_timer->stop();
|
||||
};
|
||||
m_page_2_timer->on_timeout = [&]() {
|
||||
if (m_page_2_progress_value < 100)
|
||||
m_page_2_progress_value++;
|
||||
m_page_2_progressbar->set_value(m_page_2_progress_value);
|
||||
|
||||
// Go to final page on progress completion
|
||||
if (m_page_2_progress_value == 100) {
|
||||
m_page_2_progress_value = 0;
|
||||
replace_page(*m_back_page);
|
||||
}
|
||||
};
|
||||
// Don't set a on_next_page handler for page 2 as we automatically navigate to the final page on progress completion
|
||||
|
||||
// Create the back cover
|
||||
|
|
|
@ -326,9 +326,9 @@ static bool prompt_to_stop_profiling(pid_t pid, DeprecatedString const& process_
|
|||
auto& timer_label = widget->add<GUI::Label>("...");
|
||||
Core::ElapsedTimer clock;
|
||||
clock.start();
|
||||
auto update_timer = Core::Timer::construct(100, [&] {
|
||||
auto update_timer = Core::Timer::create_repeating(100, [&] {
|
||||
timer_label.set_text(DeprecatedString::formatted("{:.1} seconds", static_cast<float>(clock.elapsed()) / 1000.0f));
|
||||
});
|
||||
}).release_value_but_fixme_should_propagate_errors();
|
||||
update_timer->start();
|
||||
|
||||
auto& stop_button = widget->add<GUI::Button>("Stop");
|
||||
|
|
|
@ -33,16 +33,9 @@ PlaybackManager::PlaybackManager(Core::Object& event_handler, NonnullOwnPtr<Demu
|
|||
, m_selected_video_track(video_track)
|
||||
, m_decoder(move(decoder))
|
||||
, m_frame_queue(make<VideoFrameQueue>())
|
||||
, m_present_timer(Core::Timer::construct())
|
||||
, m_decode_timer(Core::Timer::construct())
|
||||
{
|
||||
m_present_timer->set_single_shot(true);
|
||||
m_present_timer->set_interval(0);
|
||||
m_present_timer->on_timeout = [&] { update_presented_frame(); };
|
||||
|
||||
m_decode_timer->set_single_shot(true);
|
||||
m_decode_timer->set_interval(0);
|
||||
m_decode_timer->on_timeout = [&] { on_decode_timer(); };
|
||||
m_present_timer = Core::Timer::create_single_shot(0, [&] { update_presented_frame(); }).release_value_but_fixme_should_propagate_errors();
|
||||
m_decode_timer = Core::Timer::create_single_shot(0, [&] { on_decode_timer(); }).release_value_but_fixme_should_propagate_errors();
|
||||
}
|
||||
|
||||
void PlaybackManager::set_playback_status(PlaybackStatus status)
|
||||
|
|
|
@ -155,10 +155,10 @@ private:
|
|||
NonnullOwnPtr<VideoFrameQueue> m_frame_queue;
|
||||
Optional<FrameQueueItem> m_next_frame;
|
||||
|
||||
NonnullRefPtr<Core::Timer> m_present_timer;
|
||||
RefPtr<Core::Timer> m_present_timer;
|
||||
unsigned m_decoding_buffer_time_ms = 16;
|
||||
|
||||
NonnullRefPtr<Core::Timer> m_decode_timer;
|
||||
RefPtr<Core::Timer> m_decode_timer;
|
||||
|
||||
u64 m_skipped_frames;
|
||||
};
|
||||
|
|
|
@ -948,12 +948,12 @@ void WindowFrame::handle_menu_mouse_event(Menu& menu, MouseEvent const& event)
|
|||
void WindowFrame::start_flash_animation()
|
||||
{
|
||||
if (!m_flash_timer) {
|
||||
m_flash_timer = Core::Timer::construct(100, [this] {
|
||||
m_flash_timer = Core::Timer::create_repeating(100, [this] {
|
||||
VERIFY(m_flash_counter);
|
||||
invalidate_titlebar();
|
||||
if (!--m_flash_counter)
|
||||
m_flash_timer->stop();
|
||||
});
|
||||
}).release_value_but_fixme_should_propagate_errors();
|
||||
}
|
||||
m_flash_counter = 8;
|
||||
m_flash_timer->start();
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue