Developer forums (C::B DEVELOPMENT STRICTLY!) > CodeCompletion redesign

clangd_client string name inside the codecompletion\parser\cclogger.cpp file

<< < (3/3)

ollydbg:
Hi, Pecan, I think the variable/field name are different?

The code I see is:


--- Code: ---// ----------------------------------------------------------------------------
void CCLogger::DebugLog(const wxString& msg, int id)
// ----------------------------------------------------------------------------
{
    // Could crash here; should check if shutting down
    if (Manager::IsAppShuttingDown())
        return;

    if (!m_Parent || m_DebugLogId<1) return;

    bool debugLogging = false;
    if (m_pCfgMgr) debugLogging = m_pCfgMgr->ReadBool("/logPluginDebug_check", false);
    if ((not debugLogging) and (id == m_DebugLogId)) return;

--- End code ---

Note, the name is "logPluginDebug_check"

But in the CheckBox, the name is "CCDebugLogging".

Are they the same?

When I enabled that CheckBox in the CodeCompletion plugin's setting, I still got the code check "debugLogging" is false.

ollydbg:
The code about CCLogger is that some log functions can be called either in the worker thread or in the main gui thread.

So, you can see there are some thread event send to the main thread.

There are too many lockers in the CC code, I think we should take some time to clean up those lockers.

Pecan:
Thanks for trying. I'll restore CCLogger to the rev before I made any changes.

Pecan:
@ollydbg
I've reverted CodeCompletion.cpp/h lines that I changed related to CCLogger. I change no other lines of code.

In CCLogger I reverted to rev 13427, so you may have to change 'NativeParser' name back to ParseManager.
I recompiled CB and got no errors.
I hope I didn't miss anything. If so, let me know.

Thanks for being so patient about this.

 

ollydbg:

--- Quote from: Pecan on March 07, 2024, 06:04:56 am ---@ollydbg
I've reverted CodeCompletion.cpp/h lines that I changed related to CCLogger. I change no other lines of code.

In CCLogger I reverted to rev 13427, so you may have to change 'NativeParser' name back to ParseManager.
I recompiled CB and got no errors.
I hope I didn't miss anything. If so, let me know.

Thanks for being so patient about this.

--- End quote ---

Thanks.

BTW: I fixed  the build error for CCTest project in revision 13489.

Navigation

[0] Message Index

[*] Previous page