U++ forum: Welcome to the forum

Search on this site

Search in forums

Home » U++ TheIDE » U++ TheIDE: Other Features Wishlist and/or Bugs » [Featur+Patch]theide: list Errors first, then Warning after

Show: Today's Messages :: Show Polls :: Message Navigator
E-mail to friend

[Featur+Patch]theide: list Errors first, then Warning after [message #54766]

Wed, 09 September 2020 18:46

omari
Messages: 276
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 299 times)

regards
omari.

Report message to a moderator

Re: [Featur+Patch]theide: list Errors first, then Warning after [message #54767 is a reply to message #54766]

Wed, 09 September 2020 19:51

Klugier
Messages: 1106
Registered: September 2012
Location: Poland, Kraków

Senior 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

U++ - 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

koldo
Messages: 3453
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

Report message to a moderator

Re: [Featur+Patch]theide: list Errors first, then Warning after [message #54773 is a reply to message #54767]

Thu, 10 September 2020 10:33

omari
Messages: 276
Registered: March 2010

Experienced Member

Klugier wrote on Wed, 09 September 2020 18:51

Hello omari,

You should fix both 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.

Report message to a moderator

Re: [Featur+Patch]theide: list Errors first, then Warning after [message #54774 is a reply to message #54766]

Thu, 10 September 2020 15:42

mirek
Messages: 14271
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.

Report message to a moderator

Re: [Featur+Patch]theide: list Errors first, then Warning after [message #54784 is a reply to message #54774]

Mon, 14 September 2020 17:06

koldo
Messages: 3453
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 623 times)

Best regards
Iñaki

Report message to a moderator

Re: [Featur+Patch]theide: list Errors first, then Warning after [message #54785 is a reply to message #54784]

Mon, 14 September 2020 17:36

mirek
Messages: 14271
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:

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

Report message to a moderator

Re: [Featur+Patch]theide: list Errors first, then Warning after [message #54786 is a reply to message #54785]

Mon, 14 September 2020 21:50

koldo
Messages: 3453
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 530 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

mirek
Messages: 14271
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.

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

Mirek

Report message to a moderator

Re: [Featur+Patch]theide: list Errors first, then Warning after [message #54788 is a reply to message #54787]

Tue, 15 September 2020 09:06

koldo
Messages: 3453
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

Report message to a moderator

Previous Topic:	[solved]TheIde will quit unexpectedly when debugging
Next Topic:	First view is too complicated?

Goto Forum:

-=] Back to Top [=-

[ Syndicate this forum (XML) ] [

] [

]

Current Time: Tue Nov 04 01:05:44 CET 2025

Total time taken to generate the page: 0.04928 seconds