Skip to content

Add resume_timer usage example#724

Open
himanshigaba22 wants to merge 2 commits intothoth-tech:mainfrom
himanshigaba22:resume-timer-usage-example
Open

Add resume_timer usage example#724
himanshigaba22 wants to merge 2 commits intothoth-tech:mainfrom
himanshigaba22:resume-timer-usage-example

Conversation

@himanshigaba22
Copy link
Copy Markdown

Description

Adds usage example for the resume_timer function from SplashKit Timers API.

Type of Change

  • New usage example

Usage Example Details

  • Function: resume_timer
  • Category: Timers
  • Files added:
    • resume_timer-1-example.cpp
    • resume_timer-1-example-top-level.cs
    • resume_timer-1-example-oop.cs
    • resume_timer-1-example.py
    • resume_timer-1-example.txt
    • resume_timer-1-example.png

Checklist

  • All 6 files created with correct names
  • Code tested and runs correctly
  • Screenshot shows output only
  • Files placed in correct folder

@netlify
Copy link
Copy Markdown

netlify bot commented Apr 6, 2026

Deploy Preview for splashkit failed.

Name Link
🔨 Latest commit b2a23ca
🔍 Latest deploy log https://app.netlify.com/projects/splashkit/deploys/69d52da8d4477a0008437654

Copy link
Copy Markdown

@kottochii kottochii left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I suggest the auto-generated parts in the JSON files.

Also, please see the resume_timer's C# examples for the suggested changes (referring to the ambiguity of the file).

static void Main(string[] args)
{
// Create and start a timer
Timer myTimer = SplashKit.CreateTimer("my_timer");
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use of Timer without preceding SplashKitSDK.Timer causes ambiguity with the system class of a similar name.

using static SplashKitSDK.SplashKit;

// Create and start a timer
Timer myTimer = CreateTimer("my_timer");
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ambiguity with system's Timer

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove this file, as it is not related to the commit.

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only leave the changes directly related to the functions you are committing.

Copy link
Copy Markdown
Author

@himanshigaba22 himanshigaba22 Apr 7, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi! I have addressed all the review comments:

  • Fixed Timer ambiguity in both C# files using SplashKitSDK.Timer
  • The replace_all files and JSON changes appearing in this PR
    are from my previous replace_all PR which hasn't been merged
    into thoth-tech:main yet. My branch itself only contains
    resume_timer files — there are no unrelated changes in my branch.

Please let me know if anything else needs changing. Thank you!

@himanshigaba22 himanshigaba22 force-pushed the resume-timer-usage-example branch from adf51be to e6c6d0e Compare April 7, 2026 16:07
@himanshigaba22 himanshigaba22 force-pushed the resume-timer-usage-example branch from e6c6d0e to b2a23ca Compare April 7, 2026 16:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants