Bug 313007 - python syntax identifies strings as comments
Summary: python syntax identifies strings as comments
Status: RESOLVED FIXED
Alias: None
Product: kate
Classification: Applications
Component: syntax (show other bugs)
Version: Git
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: KWrite Developers
URL:
Keywords:
: 314773 320813 (view as bug list)
Depends on:
Blocks:
 
Reported: 2013-01-10 10:46 UTC by edA-qa mort-ora-y
Modified: 2014-05-12 20:33 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description edA-qa mort-ora-y 2013-01-10 10:46:14 UTC
The syntax file for Python now identifies normal strings as comments. I think this might have been a change to try and identify function docs, but it over-identifies other strings as comments.


Reproducible: Always

Steps to Reproduce:
Create a simple pyhton file like:

foo = bar([
  'one',
  'two'
  ])

'one' and 'two' are highligted as though they are comments. They should not be since they are just normal strings here.
Comment 1 Tomasz Narloch 2013-02-02 05:28:45 UTC
This is duplicate of 188115.

Please look at https://bugs.kde.org/show_bug.cgi?id=188115.
Patch is uploaded and it is works.
Comment 2 edA-qa mort-ora-y 2013-02-02 06:09:22 UTC
I just updated from git and the problem is still there.
Comment 3 Tomasz Narloch 2013-02-02 06:29:37 UTC
Sorry bu my mistake. 
The patch has not been committed yet.

It is available on https://bugs.kde.org/show_bug.cgi?id=188115 (attachment: second patch for python.xml in version 2.13)
or directly on:
https://bugs.kde.org/attachment.cgi?id=75965
Comment 4 Dominik Haumann 2013-03-06 20:25:40 UTC
*** Bug 314773 has been marked as a duplicate of this bug. ***
Comment 5 Dominik Haumann 2013-03-06 20:28:49 UTC
When you downoad the newest version through the download dialog in the config settings to get the version committed in https://bugs.kde.org/show_bug.cgi?id=188115#c18 , does it work then? Or is there still a patch needed?
Comment 6 edA-qa mort-ora-y 2013-03-06 20:50:07 UTC
I checked out from GIT again and did a build install. I get the newest version now and the docstrings are gone. Perhaps there is a bug in the build system that doesn't always copy the new syntax files (a different issue of course though).

Note the install button also appears not to work (it does nothing), but again, a different issue.
Comment 7 Gerald Senarclens de Grancy 2014-05-12 20:33:07 UTC
*** Bug 320813 has been marked as a duplicate of this bug. ***