Review Request 114692: Better Plotter2D
Percy Camilo Triveño Aucahuasi
percy.camilo.ta at gmail.com
Tue Dec 31 18:15:28 UTC 2013
Hi,
On 31/12/13 04:48, Albert Astals Cid wrote:
> Please use reviewboard to answer reviewboard emails, otherwise following the
> whole discussion is impossible.
>
I thought that my replies will go directly to rb site too. In a minute,
I'll reupload my comments on rb then, no problem. Apologies for that Albert.
> El Dimarts, 31 de desembre de 2013, a les 01:04:53, Percy Camilo Triveño
> Aucahuasi va escriure:
>> Hi,
>>
>> On 29/12/13 19:45, Aleix Pol Gonzalez wrote:
>>> analitzaplot/plottingenums.h
>>> <https://git.reviewboard.kde.org/r/114692/diff/1/?file=227428#file227428li
>>> ne37> (Diff revision 1)
>>>
>>> enum CoordinateSystem {
>>>
>>>
>>>
>>> 35
>>>
>>> enum CoordinateSystem {
>>>
>>> 37
>>>
>>> enum CoordinateSystem
>>>
>>> Unrelated changes, please don't commit those brace movements
>>
>> No, that is a good change, we need to have uniform code style, please
>> don't comment about non important aspects of the patch.
>
> That is unrelated, noone is saying uniform code style is bad, what Aleix is
> saying is that if you mix styling changes with code changes in a single review
> it makes it much harder to review.
>
git diff analitzaplot/plottingenums.h
give me only 4 changes about corrections in the code style, I don't know
if that is very hard to manage it, but just for the sake of don't waste
more time I will avoid to commit those changes and create another review
(yes invest more time) just for that few changes.
> Cheers,
> Albert
>
Percy
More information about the kde-edu
mailing list