Details
Details
- Reviewers
aacid albertvaka - Group Reviewers
KDE Connect - Commits
- R224:214950b3bbb4: Make sure there's not a path within the filename
Diff Detail
Diff Detail
- Repository
- R224 KDE Connect
- Branch
- master
- Lint
No Linters Available - Unit
No Unit Test Coverage
plugins/share/shareplugin.cpp | ||
---|---|---|
69 | 0 as a QString? |
plugins/share/shareplugin.cpp | ||
---|---|---|
69 | Woa.. x'D Well I guess it technically translates to (const char*) nullptr? |
plugins/share/shareplugin.cpp | ||
---|---|---|
65–71 | Make this function static. Also, I think you want to return filename instead of QString(). If I understand correctly, if there is no '/' at all in the filename, now it returns empty string. |
plugins/share/shareplugin.cpp | ||
---|---|---|
65–71 | Oh, it's not a member function. Ignore the static thing. |