[ 
https://issues.apache.org/jira/browse/DAEMON-407?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16918393#comment-16918393
 ] 

Yoshinobu Makimoto commented on DAEMON-407:
-------------------------------------------

Also, if Logging Level not found (case of default) in registry, prunmgr 
displays "Warn" because of prunmgr.c#962
{code:java}
                 else
                     ComboBox_SetCurSel(GetDlgItem(hDlg, IDC_PPLGLEVEL), 1);
{code}

> Prunmgr displays incorrect logging level.
> -----------------------------------------
>
>                 Key: DAEMON-407
>                 URL: https://issues.apache.org/jira/browse/DAEMON-407
>             Project: Commons Daemon
>          Issue Type: Bug
>          Components: Procrun
>    Affects Versions: 1.2.0
>            Reporter: Yoshinobu Makimoto
>            Priority: Minor
>
> After installing Tomcat8, Logging Level is registered as "Info" in registry. 
> But Tomcat8w(prunmgr) displays Level "Warn" on Logging tab.
> There is a mismatch of LOGL_xxx order in prunmgr.c#945-960
> {code:java}
>                   ComboBox_AddStringW(GetDlgItem(hDlg, IDC_PPLGLEVEL), 
> LOGL_ERROR);
>                   ComboBox_AddStringW(GetDlgItem(hDlg, IDC_PPLGLEVEL), 
> LOGL_WARN);
>                   ComboBox_AddStringW(GetDlgItem(hDlg, IDC_PPLGLEVEL), 
> LOGL_INFO);
>                   ComboBox_AddStringW(GetDlgItem(hDlg, IDC_PPLGLEVEL), 
> LOGL_DEBUG);
>                   if ((b = apxRegistryGetStringW(hRegserv, 
> APXREG_PARAMSOFTWARE,
>                                                  _s_log, L"Level")) != NULL) {
>                       if (!lstrcmpiW(b, LOGL_ERROR))
>                           ComboBox_SetCurSel(GetDlgItem(hDlg, IDC_PPLGLEVEL), 
> 0);
>                       else if (!lstrcmpiW(b, LOGL_INFO))
>                           ComboBox_SetCurSel(GetDlgItem(hDlg, IDC_PPLGLEVEL), 
> 1);
>                       else if (!lstrcmpiW(b, LOGL_WARN))
>                           ComboBox_SetCurSel(GetDlgItem(hDlg, IDC_PPLGLEVEL), 
> 2);
>                       else
>                           ComboBox_SetCurSel(GetDlgItem(hDlg, IDC_PPLGLEVEL), 
> 3);
>                       apxFree(b);
>                   }
> {code}
> ComboBox_AddStringW's orderĀ  is LOGL_WARN, LOGL_INFO, but 
> ComboBox_SetCurSel's order is LOGL_INFO, LOG_WARN.



--
This message was sent by Atlassian Jira
(v8.3.2#803003)

Reply via email to