[Kde-pim] Review Request 119576: how to protect a command from a change by using astyle

[Kevin Krammer]

> On Aug. 4, 2014, 1:20 nachm., Dan Vrátil wrote:
> > Sorry, this is not the right solution - there are many other places in Akonadi where we use X-AKAPPEND that could potentially get broken by astyle and we can't just workaround everything by making it a string. Altering the code just so that astyle understands it is not the right approach :)
> > 
> > As I mentioned in my email to you, next time I just have to be more careful when reviewing the changes.

Are you sure?
Here it is a string, just that the quotes are added by the macro.

Do we have any code that uses a character that looks like a minus but isn't and is not inside a string literal?


- Kevin


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119576/#review63774
-----------------------------------------------------------


On Aug. 2, 2014, 5:03 nachm., Guy Maurel wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/119576/
> -----------------------------------------------------------
> 
> (Updated Aug. 2, 2014, 5:03 nachm.)
> 
> 
> Review request for KDEPIM and Dan Vrátil.
> 
> 
> Repository: akonadi
> 
> 
> Description
> -------
> 
> astyle cannot (yet) distinguite a hythen from a minus character.
> Using a "string" such as X-AKAPPEND in the macro MAKE_CMD_ROW is a problem.
> astyle surounds the - with spaces.
> I propose to use a second macro MAKE_CMD_ROW2 providing the string as "X-AKAPPEND".
> It might be prettier to do this for all the commands.
> 
> The problem could also happens for the commands:
> X-AKLIST and X-AKLSUB
> 
> 
> Diffs
> -----
> 
>   server/tests/unittest/handlertest.cpp 6fc5d66 
> 
> Diff: https://git.reviewboard.kde.org/r/119576/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Guy Maurel
> 
>

_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/