Typo in the strports docstring

[nkvoll]

 http://twistedmatrix.com/documents/current/api/twisted.application.strports.html#service

Currently it says:

    default	Do not use this parameter. It is deprecated since Twisted

    0.

[thijs]

This looks like an issue with pydoctor or something to me..

[jesstess]

This is an epydoc bug; the "10.2.0." on a line by itself is matching on a regex for if something is a bullet. I've opened  https://sourceforge.net/tracker/?func=detail&aid=3503223&group_id=32455&atid=405618 for this issue.

Here's the minimal reproducer from the epydoc ticket:

from epydoc.markup import epytext

docstring = """                                                                                                                                                                                                  
    Return the service corresponding to a description.                                                                                                                                                           
                                                                                                                                                                                                                 
    @param default: Do not use this parameter.  It is deprecated since Twisted                                                                                                                                   
        10.2.0.                                                                                                                                                                                                  
"""
print epytext.parse_docstring(docstring, [])

which produces

<epytext><para>Return the service corresponding to a description.</para><fieldlist><field><tag>param</tag><arg>default</arg><para inline=True>Do not use this parameter.  It is deprecated since Twisted</para><olist start='0'><li bullet='10.2.0.'></li></olist></field></fieldlist></epytext>

epydoc doesn't seem to be maintained anymore, so if I don't hear back on the ticket soon the easiest thing to do here is to tweak the line a bit to not match the buggy regex.

[itamar]

Thanks for the patch! There are a couple of minor problems:

1. While that does look like it would solve the problem, it also violates our coding standard line length limit ( http://twistedmatrix.com/documents/current/core/development/policy/coding-standard.html#auto5); maybe keep the lines wrapped, but rephrase the sentence somehow so "10.2.0" isn't the last thing on the line?
2. Patches should include news files, as documented here - http://twistedmatrix.com/trac/wiki/ReviewProcess#Newsfiles - in this case you'd want an empty .misc file since it's a trivial change. If you svn add the news file it will show up in svn diff output.

Please add a new patch and re-add the review keyword and we can get this merged.

[thijs]

Thanks I will apply the patch.

[thijs]

(In [34127]) Apply strportspatch.patch: Correct formatting in the docstring of t.application.strports.service.

Author: gdeng Reviewer: jesstess, itamar, thijs Fixes: #5055