Re: Review Request: Animation Tool Docker

2012-08-19 Thread Paul Mendez
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106093/ --- (Updated Aug. 19, 2012, 10:23 p.m.) Review request for Calligra. Changes

Re: Review Request: add commands for chart operations

2012-08-19 Thread Inge Wallin
> On Aug. 19, 2012, 9:59 p.m., Inge Wallin wrote: > > Yes, I agree with Boemann. This is fine and ok to commit after you fix the > > one thing that I commented on below. Hmm, seems that review board removed my comment. It was that the coding standards demands spaces around operators. There are

Re: Review Request: add commands for chart operations

2012-08-19 Thread Inge Wallin
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106070/#review17725 --- Ship it! Yes, I agree with Boemann. This is fine and ok to com

Re: warning: overflow in implicit constant conversion in KoShape.cpp

2012-08-19 Thread C. Boemann
On Sunday 19 August 2012 15:40:25 André Wöbbeking wrote: > On Sunday 19 August 2012 11:38:52 Boudewijn Rempt wrote: > > On Saturday 18 August 2012 Aug, André Wöbbeking wrote: > > > P.S.: Please CC me. > > > > Like this? I don't get the warnings myself so I am not sure. > > Don't know whether I ge

Re: warning: overflow in implicit constant conversion in KoShape.cpp

2012-08-19 Thread André Wöbbeking
On Sunday 19 August 2012 11:38:52 Boudewijn Rempt wrote: > On Saturday 18 August 2012 Aug, André Wöbbeking wrote: > > P.S.: Please CC me. > > Like this? I don't get the warnings myself so I am not sure. Don't know whether I get it because I'm using GCC 4.7 or due to some flags. > diff --git a/li

Re: Review Request: use QFormLayout for general document/grid/misc config pages, Karbon interface config page and Sheets Open/Save config page

2012-08-19 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106084/#review17724 --- This review has been submitted with commit bba689de5fd185c90ec

Re: Review Request: add commands for chart operations

2012-08-19 Thread C. Boemann
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106070/#review17722 --- >From my point of view it looks good to go. - C. Boemann On

Re: Review Request: use QFormLayout for general document/grid/misc config pages, Karbon interface config page and Sheets Open/Save config page

2012-08-19 Thread C. Boemann
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106084/#review17721 --- Ship it! Ship It! - C. Boemann On Aug. 19, 2012, 11:16 a.m.

Review Request: Animation Tool Docker

2012-08-19 Thread Paul Mendez
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106093/ --- Review request for Calligra. Description --- GSoC work on Animation T

Review Request: Animation Tool Docker

2012-08-19 Thread Paul Mendez
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106091/ --- Review request for Calligra. Description --- GSoC work on Animation T

Re: Review Request: Introduce Author into the Calligra family

2012-08-19 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106064/#review17716 --- This review has been submitted with commit 29d5bb338ad1d6fd899

Re: Review Request: Introduce Author into the Calligra family

2012-08-19 Thread José Luis Vergara
> On Aug. 19, 2012, 1:50 p.m., José Luis Vergara wrote: > > I just turned this patch into a branch called > > author-testpatch-pentalis > > So others can test. I will review in a moment, after I finish testing. I tested: The branch compiled correctly -- check calligraauthor and calligrawords sto

Re: Review Request: Introduce Author into the Calligra family

2012-08-19 Thread José Luis Vergara
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106064/#review17714 --- Ship it! Ship It! - José Luis Vergara On Aug. 19, 2012, 6:2

Re: Review Request: Introduce Author into the Calligra family

2012-08-19 Thread José Luis Vergara
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106064/#review17710 --- I just turned this patch into a branch called author-testpatch-

Re: Review Request: Introduce Author into the Calligra family

2012-08-19 Thread Inge Wallin
> On Aug. 19, 2012, 1:08 p.m., Arjen Hiemstra wrote: > > Two things glare at me from this patch: > > > > - Why put it into Words' directory instead of its own top level? The goal > > for this is - as I understand it - to become a separate application, so why > > not start off the right way? If

Re: Review Request: Introduce Author into the Calligra family

2012-08-19 Thread Arjen Hiemstra
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106064/#review17706 --- Two things glare at me from this patch: - Why put it into Word

Re: 2.6 Release Schedule

2012-08-19 Thread Inge Wallin
On Sunday, August 19, 2012 12:50:18 Cyrille Berger Skott wrote: > On Sunday 19 Aug 2012, Inge Wallin wrote: > > On Tuesday, August 07, 2012 10:10:14 Cyrille Berger Skott wrote: > > > Hi, > > > > > > Here is the suggested schedule for 2.6: > > > > > > September 7th alpha tagging > > > September 28

Re: 2.6 Release Schedule

2012-08-19 Thread Cyrille Berger Skott
On Sunday 19 Aug 2012, Inge Wallin wrote: > On Tuesday, August 07, 2012 10:10:14 Cyrille Berger Skott wrote: > > Hi, > > > > Here is the suggested schedule for 2.6: > > > > September 7th alpha tagging > > September 28th branch+beta tagging > > October 19th RC tagging > > > > Release expected ear

Review Request: use QFormLayout for general document/grid/misc config pages, Karbon interface config page and Sheets Open/Save config page

2012-08-19 Thread Friedrich W. H. Kossebau
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106084/ --- Review request for Calligra. Description --- QFormLayout slightly imp

Re: warning: overflow in implicit constant conversion in KoShape.cpp

2012-08-19 Thread Thorsten Zachmann
On Sunday 19 August 2012 11:38:52 Boudewijn Rempt wrote: > +uint zIndex : 16; // keep maxZIndex in sync! > +uint runThrough : 16; I'm quite sure these values can get negative I think he only talked about int :1 values Thorsten ___ calligra-de

Re: warning: overflow in implicit constant conversion in KoShape.cpp

2012-08-19 Thread Boudewijn Rempt
On Saturday 18 August 2012 Aug, André Wöbbeking wrote: > Hi, > > when compiling Calligra I get: > > calligra/libs/flake/KoShape.cpp: In constructor > 'KoShapePrivate::KoShapePrivate(KoShape*)': > calligra/libs/flake/KoShape.cpp:105:48: warning: overflow in implicit > constant conversion [-Wover

Calligra 2.5 release and translation branches

2012-08-19 Thread Yuri Chornoivan
Hi, Are there any plans to switch stable branch of Calligra translations to 2.5 and enable translation for master/2.6? Thanks in advance for your answers. Best regards, Yuri ___ calligra-devel mailing list calligra-devel@kde.org https://mail.kde.or

Re: 2.6 Release Schedule

2012-08-19 Thread Inge Wallin
On Tuesday, August 07, 2012 10:10:14 Cyrille Berger Skott wrote: > Hi, > > Here is the suggested schedule for 2.6: > > September 7th alpha tagging > September 28th branch+beta tagging > October 19th RC tagging > > Release expected early November I have a problem with this schedule. Calligra 2.

Re: Review Request: Rmember the user details in Google Docs plugin

2012-08-19 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/105988/#review17693 --- This review has been submitted with commit eed23b52711243bee7e