添加自动循环播放的功能测试循环播放HEVC 4K视频会报错的问题 #67
Conversation
| @@ -508,6 +511,7 @@ BOOL CplayerDlg::PreTranslateMessage(MSG *pMsg) | |||
|
|
|||
| void CplayerDlg::OnOpenFile() | |||
| { | |||
| memset(m_strUrl, '\0', MAX_PATH); | |||
There was a problem hiding this comment.
建议修改为 strcpy(m_strUrl, ""); 设置为空字符串，这样含义更加明确。
缩进也不对，请使用四个空格进行缩进
| @@ -131,6 +131,8 @@ CplayerDlg::CplayerDlg(CWnd* pParent /*=NULL*/) | |||
| m_bYoloDetect = FALSE; | |||
| m_bMouseSelFlag = FALSE; | |||
| m_nCurMouseBtns = 0; | |||
|
|
|||
| memset(m_strUrl, '\0', MAX_PATH); | |||
There was a problem hiding this comment.
建议修改为 strcpy(m_strUrl, ""); 设置为空字符串，这样含义更加明确。
缩进也不对，请使用四个空格进行缩进
| } | ||
| WideCharToMultiByte(CP_UTF8, 0, str, -1, m_strUrl, MAX_PATH, NULL, NULL); | ||
|
|
||
| if( strlen(m_strUrl) <= 0 ){ |
There was a problem hiding this comment.
你这代码只是修改了对 m_strURL 为空字符串的判断，并没有实现自动播放？
There was a problem hiding this comment.
第一次打开需要选择打开的文件，等文件播放完成后，会自动跳过再次选择文件的过程，直接播放上一次打开的文件
There was a problem hiding this comment.
麻烦修改为命令行传参数，否则这个改动会对 testplayer 现有功能和使用逻辑造成影响
There was a problem hiding this comment.
麻烦修改为命令行传参数，否则这个改动会对 testplayer 现有功能和使用逻辑造成影响
你能不能先帮忙debug一下代码，复现一下问题，后期我再修改一个版本吧
|
嗯嗯，我都是随便改了一下用来测试找问题的。
…------------------ 原始邮件 ------------------
发件人: "rockcarry/fanplayer" ***@***.***>;
发送时间: 2022年10月28日(星期五) 中午11:23
***@***.***>;
***@***.******@***.***>;
主题: Re: [rockcarry/fanplayer] 添加自动循环播放的功能测试循环播放HEVC 4K视频会报错的问题 (PR #67)
@rockcarry commented on this pull request.
In player-windows/testplayer/playerDlg.cpp:
> @@ -131,6 +131,8 @@ CplayerDlg::CplayerDlg(CWnd* pParent /*=NULL*/) m_bYoloDetect = FALSE; m_bMouseSelFlag = FALSE; m_nCurMouseBtns = 0; + + memset(m_strUrl, '\0', MAX_PATH);
建议修改为 strcpy(m_strUrl, ""); 设置为空字符串，这样含义更加明确。
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: ***@***.***>
|
|
命令行参数可以按照这个规范修改： |
好，能先复现一下问题吗，那个测试的视频在百度云盘那里，连接我发在issue那里了，谢谢 |
使用testPlayer循环播放HEVC 4K视频会报错