Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add studio support #739

Merged
merged 46 commits into from
Oct 10, 2023

Conversation

bluepilledgreat
Copy link
Member

@bluepilledgreat bluepilledgreat commented Oct 4, 2023

#36
#516

TODO checklist:

  • RegisterProgramSize
  • Update "RobloxPlayerBeta" references

@bluepilledgreat bluepilledgreat marked this pull request as ready for review October 4, 2023 15:42
Copy link
Contributor

@EasternBloxxer EasternBloxxer left a comment

Choose a reason for hiding this comment

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

LGTM

@duckymomo360
Copy link

+rep

@nakoyasha
Copy link
Contributor

nakoyasha commented Oct 5, 2023

Overall LGTM, but I feel like Studio and Player should be separated into their own "instances", I don't think it's the greatest idea to have both share the same settings, but that's out of the scope for this PR.

@EasternBloxxer
Copy link
Contributor

Overall LGTM, but I feel like Studio and Player should be separated into their own "instances", I don't think it's the greatest idea to have both share the same settings, but that's out of the scope for this PR.

Yeah as there are flags for player that make studio not launch

@passionfr7it
Copy link

i support !

@WaviestBalloon
Copy link

lol is that C++? Wait are you binding that with Python?! Imagine using those over Rust. As a proud Rustacean and Ferris the Crab adorer, I regret to inform you that your taste in languages sucks. This is sad. You can do better. You know how easy package and dependancy management is with Cargo? Not to mention you don’t even need a Makefile. It’s great. Dynamically typed languages need to die. There’s no other option. They just do. If you like dynamic typing, you need some help. Seriously. By using a dynamically typed and interpreted language (which means its @#*!&!@ slow!!!), you are committing genocide and harming the environment more than gas cars. Rust is fast and uses clean, renewable energy through the magic of being a language compiled with LLVM. Tired of memory bugs? You should be. Shame on you for still having them when Rust exists. Tired of being bad? Time to go to Rust. Tired of being slow because you’re not smart and your friends laugh at you? Rust is quite speedy indeed (all thanks to the big brain of the compiler). Tired of not getting off the normal way? Match statements, loops, and the compiler for Rust give the best orgasms 10/10 (completely legit). Not to mention the superiority you get to feel when you show off your superior Rust code to your inferior “friends” still using some other language. Want to get rid of malware? Rust is safe, therefore malware is noware (also completely legit). You quite honestly will forget about any other language (including English because it’s slow and unsafe). You even get to add the Rust Book and its brothers to your Bible collection alongside the Arch and Gentoo Wikis. All hail Rust. TempleOS pales in religious comparison to the faith of Rustaceans. Graydon Hoare is Jesus. Amen.

Comment on lines +240 to +258
else if (LaunchArgs[0].StartsWith("roblox-studio:"))
{
commandLine = ProtocolHandler.ParseUri(LaunchArgs[0]);
if (!commandLine.Contains("-startEvent"))
commandLine += " -startEvent www.roblox.com/robloxQTStudioStartedEvent";
isStudioLaunch = true;
}
else if (LaunchArgs[0].StartsWith("roblox-studio-auth:"))
{
commandLine = HttpUtility.UrlDecode(LaunchArgs[0]);
isStudioLaunch = true;
isStudioAuth = true;
}
else if (LaunchArgs[0] == "-ide")
{
isStudioLaunch = true;
if (LaunchArgs.Length >= 2)
commandLine = $"-task EditFile -localPlaceFile \"{LaunchArgs[1]}\"";
}
Copy link
Contributor

Choose a reason for hiding this comment

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

You can simply pass Studio -protocolString, the same can be said for player by simply providing the launch URI, as player and studio can handle the URI independently!

@pizzaboxer
Copy link
Member

Looks good, provids a great base to work with.

@pizzaboxer pizzaboxer merged commit a92ea7b into bloxstraplabs:version-2.5.3 Oct 10, 2023
3 checks passed
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.

8 participants