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

support reality #1710

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from
Open

support reality #1710

wants to merge 2 commits into from

Conversation

foxlmy
Copy link

@foxlmy foxlmy commented May 4, 2023

No description provided.

Copy link
Member

@moodyhunter moodyhunter left a comment

Choose a reason for hiding this comment

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

several changes can be made to improve the overall quality, thanks anyway.

@@ -101,7 +101,8 @@ void MainWindow::MWSetSystemProxy()
LOG("ProxyAddress: " + proxyAddress);
LOG("HTTP Port: " + QSTRN(httpPort));
LOG("SOCKS Port: " + QSTRN(socksPort));
SetSystemProxy(proxyAddress, httpPort, socksPort);
// If you need to automatically set up the system proxy function,you can enable the handle commented out below
Copy link
Member

Choose a reason for hiding this comment

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

this is not related to REALITY, please consider removing it from the PR

{
QString output = QString(process.readAllStandardOutput());
QString output = QString(process.readAllStandardOutput());
Copy link
Member

Choose a reason for hiding this comment

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

unnecessary spaces?

@@ -149,7 +149,6 @@ namespace Qv2ray::core::config
{
// Use the config path found by the checks above
SetConfigDirPath(configPath);
LOG("Using ", QV2RAY_CONFIG_DIR, " as the config path.");
Copy link
Member

Choose a reason for hiding this comment

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

is there a good reason for removing this line?

@@ -393,3 +410,25 @@ void StreamSettingsWidget::on_httpHeadersEditBtn_clicked()
json["headers"] = rJson;
stream.httpSettings.headers = HttpObject::fromJson(json).headers;
}

Copy link
Member

Choose a reason for hiding this comment

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

it seems that you didn't format your codebase with clang-format, please consider doing so.

void StreamSettingsWidget::on_spiderX_textEdited(const QString &arg1)
{
stream.realitySettings.spiderX = arg1;
}
Copy link
Member

Choose a reason for hiding this comment

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

also, add a newline before EOF

Comment on lines +3312 to +3319
<message>
<source>Short id</source>
<translation>短ID(不一定支持)</translation>
</message>
<message>
<source>SpiderX</source>
<translation>初始路径(不一定支持)</translation>
</message>
Copy link
Member

Choose a reason for hiding this comment

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

translations are done on https://crowdin.com/project/qv2ray

@moodyhunter
Copy link
Member

The overall code quality can be improved and several unnecessary changes should be avoided.
Before merging this PR, I wish someone can stand up and fix the CI :(

@foxlmy
Copy link
Author

foxlmy commented May 4, 2023

I'm not a professional C++ developer, but I'm learning it, and thanks for your rigor, I've benefited a lot. Give me time and I will fix the problems you mentioned.

@Edavolkan89
Copy link

@

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants