Re: [Yade-dev] should we drop qt4 support? + clang-format ?

2020-03-10 Thread Bruno Chareyre
On Mon, 9 Mar 2020 at 23:28, Janek Kozicki (yade) wrote: > > Since I have finished all the large modifications in master I thought > about clang-formatting it. > But maybe somebody else has some ongoing large changes and prefers to wait > with this? > Hi, Do you mean to reformat the whole

Re: [Yade-dev] should we drop qt4 support?

2020-03-10 Thread Bruno Chareyre
Hi there, On Mon, 9 Mar 2020 at 21:53, Anton Gladky wrote: > > > If we decided to remove qt4, then we need to decide which version of > > GLViewer::postSelection(…) > > If we drop qt4 directory completely, then we will have only qt5-version > of this method, right? > That's what puzzled me in

Re: [Yade-dev] should we drop qt4 support? + clang-format ?

2020-03-10 Thread Vasileios Angelidakis (PGR)
Hi all, I don’t have any objections if you clang-format the whole repository. Can do the same with my local WIP repository to avoid conflicts. All the best, Vasileios From: Yade-dev on behalf of Bruno Chareyre Sent: Tuesday, March 10, 2020 12:13:47 PM To:

Re: [Yade-dev] should we drop qt4 support?

2020-03-10 Thread Janek Kozicki (yade)
Bruno Chareyre said: (by the date of Tue, 10 Mar 2020 20:42:08 +0100) > Wow, thanks a ton for clarifying. Recently I was wondering why dragging > with mouse did not work like before... I'll check. OK, great! Please push the fix to https://gitlab.com/yade-dev/trunk/-/merge_requests/438 :-))

Re: [Yade-dev] should we drop qt4 support?

2020-03-10 Thread Janek Kozicki (yade)
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

Re: [Yade-dev] should we drop qt4 support? + clang-format ?

2020-03-10 Thread Janek Kozicki (yade)
Bruno Chareyre said: (by the date of Tue, 10 Mar 2020 13:13:47 +0100) > Do you mean to reformat the whole repository? yes, exactly. Sorry for so many merge requests (again). This time I was thinking that maybe someone was working in ./core and not in ./lib directory. Maybe I should have done

Re: [Yade-dev] should we drop qt4 support?

2020-03-10 Thread Bruno Chareyre
Wow, thanks a ton for clarifying. Recently I was wondering why dragging with mouse did not work like before... I'll check. Thanks Bruno Le mar. 10 mars. 2020 20:28, Janek Kozicki (yade) a écrit : > Bruno Chareyre said: (by the date of Tue, 10 Mar 2020 13:10:24 +0100) > > That's what puzzled