Author Topic: The 08 October 2018 build (11499) is out.  (Read 39364 times)

Offline New Pagodi

  • Multiple posting newcomer
  • *
  • Posts: 41
Re: The 08 October 2018 build (11499) is out.
« Reply #30 on: October 22, 2018, 10:41:33 pm »
I'll try to use the 15 August 2018 build for a few days and see if it happens.

Offline Kilmatead

  • Single posting newcomer
  • *
  • Posts: 5
Re: The 08 October 2018 build (11499) is out.
« Reply #31 on: October 22, 2018, 11:38:58 pm »
@Kilmatead: What happens if you disable the file manager plugin?
As noted by New Pagodi, this seems to have solved it.  At least after giving it a speculative 3-hour "idle test". :)

Quote from: oBFusCATed
Does this happen with the previous night build?
No... I used that build for 2 months with nary a complaint.

Offline stahta01

  • Lives here!
  • ****
  • Posts: 7582
    • My Best Post
Re: The 08 October 2018 build (11499) is out.
« Reply #32 on: October 23, 2018, 12:14:40 am »
Can now duplicate the CPU high usage very fast with nightly build.
With FileManager Enabled, right click on folder and choose refresh under the Management pane and files tab

Tim S.
C Programmer working to learn more about C++ and Git.
On Windows 7 64 bit and Windows 10 64 bit.
--
When in doubt, read the CB WiKi FAQ. http://wiki.codeblocks.org

Offline stahta01

  • Lives here!
  • ****
  • Posts: 7582
    • My Best Post
Re: The 08 October 2018 build (11499) is out.
« Reply #33 on: October 26, 2018, 04:57:03 pm »
Can now duplicate the CPU high usage very fast with nightly build.
With FileManager Enabled, right click on folder and choose refresh under the Management pane and files tab

Tim S.

I am working on building CB using MSys2 GCC with debug build option.

What gdb commands do I need to use/learn to get the information to so someone can fix this bug.

I am a gdb newbie level user; the most advanced command I have used is bt (backtrace).

Edit: Plan is to try the bt (backtrace) command unless someone gives better suggestion.

Tim S.
« Last Edit: October 26, 2018, 05:03:38 pm by stahta01 »
C Programmer working to learn more about C++ and Git.
On Windows 7 64 bit and Windows 10 64 bit.
--
When in doubt, read the CB WiKi FAQ. http://wiki.codeblocks.org

Offline oBFusCATed

  • Developer
  • Lives here!
  • *****
  • Posts: 13413
    • Travis build status
Re: The 08 October 2018 build (11499) is out.
« Reply #34 on: October 26, 2018, 07:22:48 pm »
"thread apply all bt" is better than bt, because it prints the stack for all threads.
(most of the time I ignore long posts)
[strangers don't send me private messages, I'll ignore them; post a topic in the forum, but first read the rules!]

Offline Quiss

  • Multiple posting newcomer
  • *
  • Posts: 76
Re: The 08 October 2018 build (11499) is out.
« Reply #35 on: December 19, 2018, 01:58:40 pm »
Can now duplicate the CPU high usage very fast with nightly build.
With FileManager Enabled, right click on folder and choose refresh under the Management pane and files tab

Tim S.
I've built several revisions and found that, this starts with rev11463; rev11462 is ok.

Windows 10-x64-v1809
wxWidgets3.1.1-x64 with mingw64-7.2.0

Offline oBFusCATed

  • Developer
  • Lives here!
  • *****
  • Posts: 13413
    • Travis build status
Re: The 08 October 2018 build (11499) is out.
« Reply #36 on: December 19, 2018, 02:54:22 pm »
Thanks for doing the bisect. It is obvious that this commit is wrong. I don't know where I've looked while doing the review.

Can you try if this change fixes it:
Code
diff --git a/src/plugins/contrib/FileManager/directorymonitor.cpp b/src/plugins/contrib/FileManager/directorymonitor.cpp
index e6dd9b64f..f95220d8d 100644
--- a/src/plugins/contrib/FileManager/directorymonitor.cpp
+++ b/src/plugins/contrib/FileManager/directorymonitor.cpp
@@ -370,10 +370,10 @@ public:
                 {
                     delete it->second;
                     it = m_monmap.erase(it);
+                    continue;
                 }
-                else
-                    ++it;
             }
+            ++it;
         }
         for(size_t i=0;i<update_paths.GetCount();i++)
         {

(most of the time I ignore long posts)
[strangers don't send me private messages, I'll ignore them; post a topic in the forum, but first read the rules!]

Offline Quiss

  • Multiple posting newcomer
  • *
  • Posts: 76
Re: The 08 October 2018 build (11499) is out.
« Reply #37 on: December 20, 2018, 06:37:49 am »
Yes, this patch fixes the issue. No high cpu when right click-refresh folder, also it's been opened for half an hour.