Skip to content

Bug in documentation for extended attributes #29

@tabatkins

Description

@tabatkins

From the README:

A list of extended attributes, or None. Extended attributes are stored as Constructs, those of the forms: 'identifier', 'identifier=identifier', 'identifier(ArgumentList)', 'identifier=identifier(ArgumentList)', or 'identifier(Type,Type)' are parsed. The first identifier is stored in 'Construct.attribute', the second, if present, is stored in 'Construct.name', arguments are stored in 'Construct.arguments'. Extended attributes not mathing those five forms contain a list of tokens in 'Construct.tokens' and their name is set to 'None'.

I interpreted this as saying that, if encountering [Exposed=Window], the string "Exposed" would be in .attribute, while "Window" would be in .name. I'm not sure how else to read it, actually.

This is wrong, tho - .name contains "Exposed", .attribute appears to contain the attribute itself again, and I don't know where "Window" is stored at all. (Maybe .arguments? I haven't checked yet.)

(I'm trying to resolve #27 and speced/bikeshed#1077 right now.)

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions