Review Request 123655: Clarify/fix error message

[Alexander Potashev]

> On Май 6, 2015, 1:12 п.п., Aleix Pol Gonzalez wrote:
> > To be honest, I see why you think it should be improved, but your alternative is not all that better...
> > 
> > What about just "Could not generate file '%1'"?

"Could not generate ..." is no bueno because the file already exists before extractFileMacros() is called - it is firstly copied and then filled with the specific values of variables. "Generate" usually stands for the whole process of creating a file from scratch.


- Alexander


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


On Май 6, 2015, 6:18 д.п., Alexander Potashev wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/123655/
> -----------------------------------------------------------
> 
> (Updated Май 6, 2015, 6:18 д.п.)
> 
> 
> Review request for KDevelop and Simon Wächter.
> 
> 
> Repository: kapptemplate
> 
> 
> Description
> -------
> 
> Clarify/fix error message
> 
> I'm unsure about this change because the helper function is called GeneratePage::extractFileMacros for some reason, I don't understand why "extract". Please comment.
> 
> 
> Diffs
> -----
> 
>   src/application/generatepage.cpp 66a39644c6ef71f38f615ebe6d67b2a94e86e272 
> 
> Diff: https://git.reviewboard.kde.org/r/123655/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Alexander Potashev
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20150506/a6979401/attachment.html>