[PR #200] [MERGED] Fix V3022 warnings from PVS-Studio Static Analyzer #1087

New issue

Closed

opened 2026-02-26 09:33:36 +03:00 by kerem · 0 comments

kerem commented

2026-02-26 09:33:36 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/NickeManarin/ScreenToGif/pull/200
Author: @spirifoxy
Created: 10/26/2017
Status: ✅ Merged
Merged: 11/19/2017
Merged by: @NickeManarin

Base: master ← Head: V3022_fix

📝 Commits (1)

fab02b4 Fix V3022 warnings from PVS-Studio Static Analyzer

📊 Changes

6 files changed (+7 additions, -19 deletions)

View changed files

📝 GifRecorder/Pages/GifSettings.cs (+1 -1)
📝 Other/HookTest/Util/UserActivityHook.cs (+1 -3)
📝 ScreenToGif/Controls/MoveResizeControl.cs (+1 -1)
📝 ScreenToGif/Util/ActionStack.cs (+0 -6)
📝 ScreenToGif/Windows/Other/DownloadDialog.xaml.cs (+0 -4)
📝 ScreenToGif/Windows/Other/Localization.xaml.cs (+4 -4)

📄 Description

I'm a member of the Pinguem.ru competition on finding errors in open source projects. I removed some redundant code found using PVS-Studio

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/NickeManarin/ScreenToGif/pull/200 **Author:** [@spirifoxy](https://github.com/spirifoxy) **Created:** 10/26/2017 **Status:** ✅ Merged **Merged:** 11/19/2017 **Merged by:** [@NickeManarin](https://github.com/NickeManarin) **Base:** `master` ← **Head:** `V3022_fix` --- ### 📝 Commits (1) - [`fab02b4`](https://github.com/NickeManarin/ScreenToGif/commit/fab02b4be1d003a1ed19c82be12807c2f86dadd5) Fix V3022 warnings from PVS-Studio Static Analyzer ### 📊 Changes **6 files changed** (+7 additions, -19 deletions) <details> <summary>View changed files</summary> 📝 `GifRecorder/Pages/GifSettings.cs` (+1 -1) 📝 `Other/HookTest/Util/UserActivityHook.cs` (+1 -3) 📝 `ScreenToGif/Controls/MoveResizeControl.cs` (+1 -1) 📝 `ScreenToGif/Util/ActionStack.cs` (+0 -6) 📝 `ScreenToGif/Windows/Other/DownloadDialog.xaml.cs` (+0 -4) 📝 `ScreenToGif/Windows/Other/Localization.xaml.cs` (+4 -4) </details> ### 📄 Description I'm a member of the Pinguem.ru competition on finding errors in open source projects. I removed some redundant code found using PVS-Studio --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

kerem

2026-02-26 09:33:36 +03:00

closed this issue
added the
pull-request
label

kerem referenced this issue

2026-02-26 09:34:22 +03:00

[PR #1087] [MERGED] Show hours if duration is greater than or equal to 60 minutes #1305

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

starred/ScreenToGif#1087

No description provided.

Rows
Columns