| Summary: | Execute spell-checking only on non-code parts | ||
|---|---|---|---|
| Product: | [Applications] kdevelop | Reporter: | greendeath |
| Component: | general | Assignee: | KDevelop-Devel List <kdevelop-devel> |
| Status: | RESOLVED INTENTIONAL | ||
| Severity: | wishlist | CC: | ackdrummaster, domseichter, lashack, roger.larsson, shafff, uetsah |
| Priority: | NOR | ||
| Version: | 4.0.0 | ||
| Target Milestone: | --- | ||
| Platform: | unspecified | ||
| OS: | Other | ||
| Latest Commit: | Version Fixed In: | ||
| Sentry Crash Report: | |||
|
Description
greendeath
2002-08-27 15:53:28 UTC
*** Bug 61282 has been marked as a duplicate of this bug. *** KDevelop uses kate as editor which supports spell checking of files. But they want spell checking of the strings in the i18n calls, which may or may not be supported by kate. With the new text editing stuff, duchain highlighting, etc. We should have the ability to do this for KDevelop 4. Reopening. First thing to do is provide a KTE interface for spellchecking. Changing the title to be more generic *** Bug 128713 has been marked as a duplicate of this bug. *** *** This bug has been confirmed by popular vote. *** If this is ever implemented, then - in addition to strings in i18n calls - spellchecking should also apply to doxygen comments (or maybe even comments in general). I'm not the best programmer in the world here so I cant help much coding wise, however would it be possible to have the spell check find the "" marks or the // one or */ /* and only check those things with in that? tell me what you think. thanks Fixed/Implemented in KDE 4.4 - no? Please try it out and open up designated bug reports for the glitches you find there. i dare to reopen this report. KDevelop 0.10.2 still doesn't skip code parts during Tools -> Spellcheck. I tried enabling extensions in Tools -> Configure editor, but nothing changed. I'd like to check i18n'd strings and (optionally) comments (In reply to comment #9) > Fixed/Implemented in KDE 4.4 - no? Please try it out and open up designated bug > reports for the glitches you find there. How is this fixed for KDE 4.4? I can't see a KTE interface to execute spell-checking on some predefined string. Let alone the part using duchain to find out the strings of i18n calls. But by default only string contents and comments are spellchecked. i18n() contains strings, so everything should be fine? Why would we need kde-special handling for i18n? Please convince me that this is still needed, otherwise I'll just close it. *** Bug 80865 has been marked as a duplicate of this bug. *** I just talked to Andreas: His wish was for example this:
i18n("please spell check this string");
QFile("/dont/spellcheck/this.txt");
It's simply not possible to implement this, as there is no /useful/ way to implement this. In the end a user would have to decide whether arguments to a function should get spell checked or not. Far too much effort for imo not much gain. Closing finally, as the main part (i.e. only string contents and comments get spell checked) is implemented in Kate.
(In reply to comment #14) > I just talked to Andreas: His wish was for example this: > > i18n("please spell check this string"); > QFile("/dont/spellcheck/this.txt"); Minor nitpick: It was not my wish, that was simply how I understood the report taking into account that kate already checks only comments and strings and yet the report was re-opened. I don't personally use spell-checking at all, hence have no wishes in any way related to that :) |