← Back to team overview

yade-dev team mailing list archive

Re: should we drop qt4 support?

 

Bruno Chareyre said:     (by the date of Tue, 10 Mar 2020 13:10:24 +0100)
> That's what puzzled me in the first place. So deeply that I didn't dare
> replying to the question. :)

Ah sorry, here's exactly what I meant:

qt5/GLViewer.cpp, function postSelection() was written by Anton in 2015-06-26

qt4/GLViewer.cpp, function postSelection() was changed by Bruno in 2017-05-24

hm,hm.. Bruno, did you fix something in 2017 in postSelection? Something about

   	Omega::instance().getScene()->selectedBody = -1;

And maybe at that time you were using qt4? And you were playing with moving bodies around?

If so, then maybe the "correct" version of this function is rather in qt4 directory?

> Yes I think we can drop Qt4.
> Back to Ubuntu16 Qt5 is available.

OK, great. That (first clang-format) merge request about QT does just
that. It reformats the directory and removes qt4 directory.

However currently in tha MR there is postSelection() from qt5 version
by Anton from 2015-06-26. We can change that if you want :)

best regards
Janek


PS: here's the full (clang-formatted) snippet:

qt5, Anton, year 2015:

	if (selection < 0) {
		if (last >= 0) {
			Body::byId(Body::id_t(last))->state->blockedDOFs = initBlocked;
			last                                             = -1;
			Omega::instance().getScene()->selectedBody       = -1;
		}
		if (isMoving) {
			displayMessage("Moving finished");
			mouseMovesCamera();
			isMoving                                   = false;
			Omega::instance().getScene()->selectedBody = -1;
		}
		return;
	}

and qt4, Bruno, year 2017:

	if (selection < 0) {
		Omega::instance().getScene()->selectedBody = -1;
		if (last >= 0) {
			Body::byId(Body::id_t(last))->state->blockedDOFs = initBlocked;
			last                                             = -1;
		}
		if (isMoving) {
			displayMessage("Moving finished");
			mouseMovesCamera();
			isMoving = false;
		}
		return;
	}

Bruno's code is shorter by one line and the only difference is that
selectedBody = -1; is done alaways, not just inside if(…).

Sorry about writing so much about single line. I simply compared the two dirs,
and saw just this one difference between the two :)



--
Janek Kozicki, PhD. DSc. Arch. Assoc. Prof.
Gdańsk University of Technology
Faculty of Applied Physics and Mathematics
Department of Theoretical Physics and Quantum Information
--
http://yade-dem.org/
http://pg.edu.pl/jkozicki (click English flag on top right)


Follow ups

References