Skip to content

Fix an exception in drive_virtual.cpp - #6421

Merged
joncampbell123 merged 1 commit into
joncampbell123:masterfrom
zhongjianhua163:master
Jul 20, 2026
Merged

Fix an exception in drive_virtual.cpp#6421
joncampbell123 merged 1 commit into
joncampbell123:masterfrom
zhongjianhua163:master

Conversation

@zhongjianhua163

Copy link
Copy Markdown
Contributor

我很困惑, 为什么第236行在对current_normalized_dir的尾部取值前进行了空字符串的判断,而在第235行中对头部进行取值时却没有判断空字符串??

我在首次测试中就遇到此处触发的异常。由于DEBUG版在遇到此异常时无法继续,所以我添加了空字符串的判断。

check current_normalized_dir
@joncampbell123

Copy link
Copy Markdown
Owner

Google translate:

"I'm confused. Why does line 236 check an empty string before taking a value for the current_normalized_dir's tail, but not an empty string when taking the head value in line 235? I encountered an exception triggered here during my first test. Since the DEBUG version cannot continue when encountering this exception, I added an empty string check."

@zhongjianhua163

Copy link
Copy Markdown
Contributor Author

Google translate:

"I'm confused. Why does line 236 check an empty string before taking a value for the current_normalized_dir's tail, but not an empty string when taking the head value in line 235? I encountered an exception triggered here during my first test. Since the DEBUG version cannot continue when encountering this exception, I added an empty string check."

“but not an empty string ...” -> "but not check an empty string ..." maybe, im not sure, haha

@joncampbell123
joncampbell123 merged commit e391138 into joncampbell123:master Jul 20, 2026
22 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.

2 participants