Author Topic: The 08 July 2018 build (11428) is out.  (Read 4222 times)

Offline LETARTARE

  • Regular
  • ***
  • Posts: 354
  • L'ami de l'homme.The friend of man.
Re: The 08 July 2018 build (11428) is out.
« Reply #15 on: July 20, 2018, 10:22:16 am »
for me, it's correct. (svn 11428, VISTA)
C::B  : svn11428, wxwidgets2.8,  plugins : QtPregen-2.3.0
1-Vista Business Pack 2 32bits :  TDM-GCC 4.9.2, 
2-OpenSuse-42.3-Leap 64bits :  g++-4.8.5,
=> !! The messages are translated by GOOGLE !!

Offline Miguel Gimenez

  • Multiple posting newcomer
  • *
  • Posts: 94
Re: The 08 July 2018 build (11428) is out.
« Reply #16 on: July 20, 2018, 12:19:39 pm »
The last revision that "worked" was 11375. The function was intended to jump only between errors, but as Obfuscated said:

Quote
The old version moved one line down no matter that the log contained no
errors. The new versions changes m_ErrorIndex only if a match is found.

so he fixed it. The function now behaves as expected (good) but now warnings are ignored.

A possible feature can be jumping between warnings if there are no errors.
« Last Edit: July 20, 2018, 04:41:51 pm by Miguel Gimenez »

Offline PresidentSkroob

  • Single posting newcomer
  • *
  • Posts: 5
Re: The 08 July 2018 build (11428) is out.
« Reply #17 on: July 20, 2018, 12:53:46 pm »
I understand that it was fixed, but old functionality was very useful for me.
Feature with jumping between warnings if there are no more errors would be great, at least for me:)
Thanks for explaining.
"evil will always triumph, because good is dumb"

Offline Miguel Gimenez

  • Multiple posting newcomer
  • *
  • Posts: 94
Re: The 08 July 2018 build (11428) is out.
« Reply #18 on: July 20, 2018, 01:23:38 pm »
This patch implements jumping to warnings when there are no errors. I don't post a ticket now because I don't know if developers will like this feature.
« Last Edit: July 20, 2018, 04:42:57 pm by Miguel Gimenez »

Offline oBFusCATed

  • Developer
  • Lives here!
  • *****
  • Posts: 10967
    • Travis build status
Re: The 08 July 2018 build (11428) is out.
« Reply #19 on: July 20, 2018, 07:41:51 pm »
This patch is too complex. If allow warnings we should jump to them all the time. Not only if there are no errors. The behaviour in the patch would be confusing.
(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!]