Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ext call doco warts #35739

Closed
mwhudson opened this issue Dec 14, 2001 · 8 comments
Closed

ext call doco warts #35739

mwhudson opened this issue Dec 14, 2001 · 8 comments
Assignees
Labels
docs Documentation in the Doc dir

Comments

@mwhudson
Copy link

BPO 493243
Nosy @mwhudson, @freddrake
Files
  • ref5-2.diff: another go.
  • Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.

    Show more details

    GitHub fields:

    assignee = 'https://github.com/freddrake'
    closed_at = <Date 2002-06-20.14:25:27.000>
    created_at = <Date 2001-12-14.10:22:49.000>
    labels = ['docs']
    title = 'ext call doco warts'
    updated_at = <Date 2002-06-20.14:25:27.000>
    user = 'https://github.com/mwhudson'
    

    bugs.python.org fields:

    activity = <Date 2002-06-20.14:25:27.000>
    actor = 'fdrake'
    assignee = 'fdrake'
    closed = True
    closed_date = None
    closer = None
    components = ['Documentation']
    creation = <Date 2001-12-14.10:22:49.000>
    creator = 'mwh'
    dependencies = []
    files = ['250']
    hgrepos = []
    issue_num = 493243
    keywords = []
    message_count = 8.0
    messages = ['8174', '8175', '8176', '8177', '8178', '8179', '8180', '8181']
    nosy_count = 2.0
    nosy_names = ['mwh', 'fdrake']
    pr_nums = []
    priority = 'high'
    resolution = 'fixed'
    stage = None
    status = 'closed'
    superseder = None
    type = None
    url = 'https://bugs.python.org/issue493243'
    versions = []
    

    @mwhudson
    Copy link
    Author

    Now they've been compiled <wink>, I notice that there
    are some warts in my docs for the *- and **-style call
    syntax.

    1. the "argument_list" production is really, really
      confusing. there must be a better BNF-style way of
      saying that. I don't think vertically centering the
      production name against the production helps.

    2. For some reason, where I say

    It is unusual for both keyword arguments and the
    "*expression"syntax to be used in the same call, so in
    practice this confusion does not arise.

    there's no space between "*expression"' and syntax'.
    I'd guess that this is because in the source, the
    \samp{} macro is the last thing on the line, but why
    that should lead to a missing space is beyond me --
    more latex2html bugs?

    (just noticed the same thing a bit higher up too --

    the "**expression"argument, if any

    No hurry with these.

    @mwhudson mwhudson added the docs Documentation in the Doc dir label Dec 14, 2001
    @mwhudson mwhudson added the docs Documentation in the Doc dir label Dec 14, 2001
    @freddrake
    Copy link
    Member

    Logged In: YES
    user_id=3066

    Fixed table cell alignment in Doc/perl/python.perl revision
    1.115.

    Fixed item 2: Worked around spaces problem in
    Doc/ref/ref5.tex 1.53

    This bug remains open; I still need to address the basic
    problem in item 1 (confusing pseudo-EBNF).

    @mwhudson
    Copy link
    Author

    Logged In: YES
    user_id=6656

    Another problem with the pseudo-EBNF: it's wrong. Oops.

    It suggests that f(a, **b) isn't legal, for example.

    I've attached an attempt I think it right, but I'm not sure
    and haven't compiled it (& so don't know if it looks any less
    confusing than last time).

    @mwhudson
    Copy link
    Author

    Logged In: YES
    user_id=6656

    I'm going to check my fix in unless someone complains *real*
    soon.

    At least I know it compiles now, and I think it's an
    improvement.

    @freddrake
    Copy link
    Member

    Logged In: YES
    user_id=3066

    Bumped priority; needs review.

    @freddrake
    Copy link
    Member

    Logged In: YES
    user_id=3066

    1. Do you still see vertical centering of the production
      name? That should be aligned with the top of the definition
      text. I've played contortionist with that production in
      Doc/ref/ref5.tex revision 1.62.

    2. That's not another bug, that's the one that keeps biting
      us. I'm not sure whether that's in latex2html itself or in
      my extension code for it. ;-( This has already been fixed
      on the trunk.

    I'm closing this, since I think we're ok now; feel free to
    re-open or file a new report if you think my changes aren't
    sufficient.

    @mwhudson
    Copy link
    Author

    Logged In: YES
    user_id=6656

    Sorry, you need to reread my comment of 2001-12-20: the
    problem with the pseudo-BNF as it stands is that it is
    WRONG! This is what my patch attempts to fix.

    The problems mentioned originally have been fixed -- perhaps
    I should have opened a new report.

    @freddrake
    Copy link
    Member

    Logged In: YES
    user_id=3066

    Ok, I've fixed that with a different patch. Checked in as
    Doc/ref/ref5.tex revision 1.63.

    Thanks for your patience, and persistence!

    @ezio-melotti ezio-melotti transferred this issue from another repository Apr 9, 2022
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Labels
    docs Documentation in the Doc dir
    Projects
    None yet
    Development

    No branches or pull requests

    2 participants