Overview
Examples
Screenshots
Comparisons
Applications
Download
Documentation
Tutorials
Bazaar
Status & Roadmap
FAQ
Authors & License
Forums
Funding Ultimate++
Search on this site
Search in forums












SourceForge.net Logo
Home » U++ TheIDE » U++ TheIDE: Other Features Wishlist and/or Bugs » [Featur+Patch]theide: list Errors first, then Warning after
[Featur+Patch]theide: list Errors first, then Warning after [message #54766] Wed, 09 September 2020 18:46 Go to next message
omari is currently offline  omari
Messages: 224
Registered: March 2010
Experienced Member
that allows us to find the errors without scrolling through warnings.

here the patch (generated by 'git diff').
  • Attachment: errors.diff
    (Size: 1.20KB, Downloaded 9 times)


regards
omari.
Re: [Featur+Patch]theide: list Errors first, then Warning after [message #54767 is a reply to message #54766] Wed, 09 September 2020 19:51 Go to previous messageGo to next message
Klugier is currently offline  Klugier
Messages: 704
Registered: September 2012
Location: Poland, Kraków
Contributor
Hello omari,

You should fix both Smile I encourage you to turn option on that treats warnings as errors. In this case uppsrc shouldn't produced any warnings. I hope we are on the good path to do it.

Klugier


Ultimate++ - one framework to rule them all.

[Updated on: Wed, 09 September 2020 19:52]

Report message to a moderator

Re: [Featur+Patch]theide: list Errors first, then Warning after [message #54768 is a reply to message #54766] Wed, 09 September 2020 20:35 Go to previous messageGo to next message
koldo is currently offline  koldo
Messages: 3196
Registered: August 2008
Senior Veteran
omari wrote on Wed, 09 September 2020 18:46
that allows us to find the errors without scrolling through warnings.

here the patch (generated by 'git diff').
This seems a good idea. In addition, some of the warnings, generated out of the code of our interest, are unavoidable, so at least they could be at the end.


Best regards
Iñaki
Re: [Featur+Patch]theide: list Errors first, then Warning after [message #54773 is a reply to message #54767] Thu, 10 September 2020 10:33 Go to previous messageGo to next message
omari is currently offline  omari
Messages: 224
Registered: March 2010
Experienced Member
Klugier wrote on Wed, 09 September 2020 18:51
Hello omari,

You should fix both Smile I encourage you to turn option on that treats warnings as errors. In this case uppsrc shouldn't produced any warnings. I hope we are on the good path to do it.

Klugier


Of course we fix both, but fixing errors has a higher priority than warnings Smile.


regards
omari.
Re: [Featur+Patch]theide: list Errors first, then Warning after [message #54774 is a reply to message #54766] Thu, 10 September 2020 15:42 Go to previous messageGo to next message
mirek is currently offline  mirek
Messages: 12716
Registered: November 2005
Ultimate Member
omari wrote on Wed, 09 September 2020 18:46
that allows us to find the errors without scrolling through warnings.

here the patch (generated by 'git diff').


This is a good idea (well, sort of, most of us treat warnings as errors anyway), but I think it would be even better to have an option (something like "Hide warnings") as I do not like putting them out of context.

In either case, for now I am adding to redmine, I do not want any changes until next release.
Re: [Featur+Patch]theide: list Errors first, then Warning after [message #54784 is a reply to message #54774] Mon, 14 September 2020 17:06 Go to previous messageGo to next message
koldo is currently offline  koldo
Messages: 3196
Registered: August 2008
Senior Veteran
When reading this post I was thinking in these CLANG warnings that seem unavoidable. Warnings like this are the reason why I found this proposal interesting:
index.php?t=getfile&id=6197&private=0
  • Attachment: war.png
    (Size: 12.13KB, Downloaded 88 times)


Best regards
Iñaki
Re: [Featur+Patch]theide: list Errors first, then Warning after [message #54785 is a reply to message #54784] Mon, 14 September 2020 17:36 Go to previous messageGo to next message
mirek is currently offline  mirek
Messages: 12716
Registered: November 2005
Ultimate Member
koldo wrote on Mon, 14 September 2020 17:06
When reading this post I was thinking in these CLANG warnings that seem unavoidable. Warnings like this are the reason why I found this proposal interesting:
index.php?t=getfile&id=6197&private=0


Why don't you just fix it? CLANG warns you that the case does not match... Basically, if you use something like

#include <core/core.h>

then this will work in Win32 (with warning), but will NOT work in POSIX....

Mirek
Re: [Featur+Patch]theide: list Errors first, then Warning after [message #54786 is a reply to message #54785] Mon, 14 September 2020 21:50 Go to previous messageGo to next message
koldo is currently offline  koldo
Messages: 3196
Registered: August 2008
Senior Veteran
Sorry Mirek. Those warnings are generated inside Core .cpp files.
I was not worried because in the past this was not usually considered a priority.
index.php?t=getfile&id=6198&private=0
  • Attachment: 26.png
    (Size: 146.71KB, Downloaded 61 times)


Best regards
Iñaki

[Updated on: Mon, 14 September 2020 21:53]

Report message to a moderator

Re: [Featur+Patch]theide: list Errors first, then Warning after [message #54787 is a reply to message #54786] Mon, 14 September 2020 22:47 Go to previous messageGo to next message
mirek is currently offline  mirek
Messages: 12716
Registered: November 2005
Ultimate Member
koldo wrote on Mon, 14 September 2020 21:50
Sorry Mirek. Those warnings are generated inside Core .cpp files.
I was not worried because in the past this was not usually considered a priority.
index.php?t=getfile&id=6198&private=0


Because you have wrongly setup assembly. Been there... You need to fix the case IMO...

Mirek
Re: [Featur+Patch]theide: list Errors first, then Warning after [message #54788 is a reply to message #54787] Tue, 15 September 2020 09:06 Go to previous message
koldo is currently offline  koldo
Messages: 3196
Registered: August 2008
Senior Veteran
Excellent, fixed.
In my PC, in some .var files, in some cases, the path until /upp folder appeared in lower case, when the real path has letters in upper case.
Curiously, although the warning is devoted to protect Linux like OS, the same set of folders is compiled using a Linux virtual machine without warnings....
From now on I will report warnings in U++ sources.


Best regards
Iñaki
Previous Topic: [solved]TheIde will quit unexpectedly when debugging
Next Topic: First view is too complicated?
Goto Forum:
  


Current Time: Sun Sep 27 21:30:26 CEST 2020

Total time taken to generate the page: 0.01119 seconds