Bug 708072 (test-stretchy)

Test page for stretchy and large operators

RESOLVED FIXED in mozilla12

Status

()

defect
RESOLVED FIXED
8 years ago
7 years ago

People

(Reporter: fredw, Assigned: fredw)

Tracking

Trunk
mozilla12
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

Assignee

Description

8 years ago
Posted patch PatchSplinter Review
I open this bug to create a test generated from our operator dictionary, for verifying stretchy and large operators.

Unfortunately, it seems that the dynamically generated content is not well rendered, so I 'll give a static version of the test page.
Assignee

Comment 1

8 years ago
Assignee: nobody → fred.wang
Assignee

Comment 2

8 years ago
Comment on attachment 579433 [details] [diff] [review]
Patch

I ask a review now, since fixing the issue with dynamically generated content seems out of the scope of this bug.
Attachment #579433 - Flags: review?(karlt)
Comment on attachment 579433 [details] [diff] [review]
Patch

Looks good thanks.

Some things to consider as possible future improvements:

updateOperatorDictionary.pl would be easier to follow if the array index
constants were names, or, perhaps simpler, the data could be collected into
hash tables with the keys being 'description', 'lspace', etc.

I wonder whether it would be worthwhile considering operators that are
stretchy (have a direction) but are not stretchy by default.  e.g. &Sum.
Attachment #579433 - Flags: review?(karlt) → review+
Assignee

Updated

8 years ago
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/9dfb1c2c5586
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Assignee

Updated

7 years ago
Blocks: 755541
You need to log in before you can comment on or make changes to this bug.