U++ forum: Welcome to the forum

Search on this site

Search in forums

Home » Community » U++ community news and announcements » U++ 2023.2 "pre-alpha"

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

Re: U++ 2023.2 "pre-alpha" [message #60136 is a reply to message #60126]

Wed, 13 September 2023 11:04

mirek
Messages: 13975
Registered: November 2005

Ultimate Member

luoganda wrote on Sat, 09 September 2023 10:49

Quote:
Now I am not sure whether whether you are speaking about pre-libclang homegrown parser or about difference between 2023.1 and 2023.2.

I meant pre-clang(homegrown) parser - since it's probably the only one who had "panoramic view" of all classes in Navwnd at hand all the time.

Navigator window is back now - I hoped you will check it out.

Quote:

12+ cores?
hah, if anything does not work well with 4 - you can throw it away right away...

Make a distinction between development machine and average target user machine. U++ apps work just fine on single core 1Ghz ARM CPUs. But time is money and powerful machines are for pennies today. Moreover, C++ programmers are rare and expensive. Usually one month salary of entry level C++ programmer is worth 5+ multicore workstations...

That said, my "vacation" machine is now quite old 2GHz double core notebook with Linux installed (I rarely need it anyway). It was still OK-ish experience.

Anyway, Win7 probably does not work anymore - blame clang.

Mirek

Report message to a moderator

Re: U++ 2023.2 "pre-alpha" [message #60137 is a reply to message #60130]

Wed, 13 September 2023 11:17

mirek
Messages: 13975
Registered: November 2005

Ultimate Member

Tom1 wrote on Mon, 11 September 2023 15:33

Hi Mirek!

Looks good.

Can you change the 'Main configuration' drop list to not grab keyboard focus. After changing the Main configuration, trying to start compiling with F7 will not work until I click the code editor area. This issue was introduced in 2023.1, I think, but I discovered it only after release...

Best regards,

Tom

Hopefully fixed, thanks.

Report message to a moderator

Re: U++ 2023.2 "pre-alpha" [message #60138 is a reply to message #60131]

Wed, 13 September 2023 11:34

mirek
Messages: 13975
Registered: November 2005

Ultimate Member

Tom1 wrote on Mon, 11 September 2023 15:38

And another thing...

When using "Show git history for file..." for "plugin/tif/tifupp.cpp" and comparing the previous and current version, very large region of code seems to be changed at the first glance. However, in reality, the comparison function does not match the lines very well and there are just few changes to display. Please take a look.

Best regards,

Tom

There is a change in indentation. Should we ignore it? (That is not rethrical question, I really wonder whether we should... it is doable)

Mirek

Report message to a moderator

Re: U++ 2023.2 "pre-alpha" [message #60139 is a reply to message #60137]

Wed, 13 September 2023 12:42

Tom1
Messages: 1212
Registered: March 2007

Senior Contributor

mirek wrote on Wed, 13 September 2023 12:17

Tom1 wrote on Mon, 11 September 2023 15:33
Hi Mirek!

Looks good.

Can you change the 'Main configuration' drop list to not grab keyboard focus. After changing the Main configuration, trying to start compiling with F7 will not work until I click the code editor area. This issue was introduced in 2023.1, I think, but I discovered it only after release...

Best regards,

Tom

Hopefully fixed, thanks.

Thanks! // BR, Tom

Report message to a moderator

Re: U++ 2023.2 "pre-alpha" [message #60140 is a reply to message #60138]

Wed, 13 September 2023 12:49

Tom1
Messages: 1212
Registered: March 2007

Senior Contributor

mirek wrote on Wed, 13 September 2023 12:34

Tom1 wrote on Mon, 11 September 2023 15:38
And another thing...

When using "Show git history for file..." for "plugin/tif/tifupp.cpp" and comparing the previous and current version, very large region of code seems to be changed at the first glance. However, in reality, the comparison function does not match the lines very well and there are just few changes to display. Please take a look.

Best regards,

Tom

There is a change in indentation. Should we ignore it? (That is not rethrical question, I really wonder whether we should... it is doable)

Mirek

Hi,

Yes, I see there are several indentation changes, but the real issue begins on line 904 "break;", where it completely loses sync. Then it re-synchronizes to "break;" on line 1001, within another switch. Do you get this same behavior?

I do not mind seeing the indentation changes, especially as they are so neatly presented inline. It is actually good to see why a file was updated, even if it just indentation. But the large portions of code (such as 904...) getting out of sync is a problem.

Best regards,

Tom

Report message to a moderator

Re: U++ 2023.2 "pre-alpha" [message #60141 is a reply to message #60140]

Wed, 13 September 2023 13:40

mirek
Messages: 13975
Registered: November 2005

Ultimate Member

Tom1 wrote on Wed, 13 September 2023 12:49

mirek wrote on Wed, 13 September 2023 12:34
Tom1 wrote on Mon, 11 September 2023 15:38
And another thing...

When using "Show git history for file..." for "plugin/tif/tifupp.cpp" and comparing the previous and current version, very large region of code seems to be changed at the first glance. However, in reality, the comparison function does not match the lines very well and there are just few changes to display. Please take a look.

Best regards,

Tom

There is a change in indentation. Should we ignore it? (That is not rethrical question, I really wonder whether we should... it is doable)

Mirek

Hi,

Yes, I see there are several indentation changes, but the real issue begins on line 904 "break;", where it completely loses sync. Then it re-synchronizes to "break;" on line 1001, within another switch. Do you get this same behavior?

Tom

Yes, and I believe it works correctly. That break has different indentation, so it "synchronizes" (not that it is how it really works) on the first break that has the same indent. Which is on line 1001. (Please check whether I interpret it correctly...)

Report message to a moderator

Re: U++ 2023.2 "pre-alpha" [message #60142 is a reply to message #60141]

Wed, 13 September 2023 14:43

Tom1
Messages: 1212
Registered: March 2007

Senior Contributor

mirek wrote on Wed, 13 September 2023 14:40

Tom1 wrote on Wed, 13 September 2023 12:49
mirek wrote on Wed, 13 September 2023 12:34
Tom1 wrote on Mon, 11 September 2023 15:38
And another thing...

When using "Show git history for file..." for "plugin/tif/tifupp.cpp" and comparing the previous and current version, very large region of code seems to be changed at the first glance. However, in reality, the comparison function does not match the lines very well and there are just few changes to display. Please take a look.

Best regards,

Tom

There is a change in indentation. Should we ignore it? (That is not rethrical question, I really wonder whether we should... it is doable)

Mirek

Hi,

Yes, I see there are several indentation changes, but the real issue begins on line 904 "break;", where it completely loses sync. Then it re-synchronizes to "break;" on line 1001, within another switch. Do you get this same behavior?

Tom

Yes, and I believe it works correctly. That break has different indentation, so it "synchronizes" (not that it is how it really works) on the first break that has the same indent. Which is on line 1001. (Please check whether I interpret it correctly...)

Well, I guess it's just so complicated problem that there is no easy nor perfect solution. In this specific case the indentation is the cause for this readability issue, but it could be any change really that can cause similar 'loss of sync'. Probably not worth spending tens of hours, to make it find the minimal set of changed characters in optimal sequences and visualize accordingly.

I just wanted to point out that the current comparison on this file makes up an illusion that the file is all different now. It is really hard to see what the real changes are.

Best regards,

Tom

Report message to a moderator

Re: U++ 2023.2 "pre-alpha" [message #60145 is a reply to message #60098]

Wed, 13 September 2023 19:13

jjacksonRIAB
Messages: 220
Registered: June 2011

Experienced Member

Mirek,

Since I know you're checking this thread, you might want to look into this bug report/fix for pre-alpha. It gave me back my CPU on linux.

https://www.ultimatepp.org/forums/index.php?t=msg&th=121 58&start=0&

Report message to a moderator

Re: U++ 2023.2 "pre-alpha" [message #60147 is a reply to message #60098]

Thu, 14 September 2023 02:11

zsolt
Messages: 697
Registered: December 2005
Location: Budapest, Hungary

Contributor

Could you include this one line XmlParser bugfix?
https:// www.ultimatepp.org/forums/index.php?t=msg&th=12159&s tart=0&

Report message to a moderator

Re: U++ 2023.1 "pre-alpha" [message #60157 is a reply to message #60135]

Sat, 16 September 2023 20:27

Oblivion
Messages: 1093
Registered: August 2007

Senior Contributor

Quote:

Oblivion,

It looks like Mirek really squashed it this time with his latest bugfix. I've been running it for several days now without a single issue but it would be nice to have a second confirmation when you have time.

Sorry, I'm a bit late.
I've been continuously running the patched TheIDE since last week (compiling huge apps), and I think at this point it is safe to say this annoying and nasty bug is gone (on Linux).

Thanks and well done, Mirek!

Best regards,
Oblivion

Github page: https://github.com/ismail-yilmaz
upp-components: https://github.com/ismail-yilmaz/upp-components
Bobcat the terminal emulator: https://github.com/ismail-yilmaz/Bobcat

Report message to a moderator