Missing functionality for RegExp.dotAll #2008

Our implementation of RegExp dotAll doesn't seem to have all required functionality.
As reported in Topic 28836.

Preliminary analysis shows two problems:

The s flag is parsed in strings /s but not when passed as separate flags in the Regex object CTOR. This seems to have been an oversight in my initial implementation.
The RegExp.prototype.dotall property was finalized as dotAll with a capital A (to join the rest of the inconsistent casing of RegExp properties in the standard where some have camelCase and some do not). My implementation was dotall, therefore coming up with undefined property for .dotAll because of case sensitivity.

This shouldn't be too hard to fix.

See #1284 for the initial implementation. Funny how this has not been an issue for 3 whole years until it got reported.

Our implementation of RegExp dotAll doesn't seem to have all required functionality. As reported in [Topic 28836](https://forum.palemoon.org/viewtopic.php?f=70&t=28836). Preliminary analysis shows two problems: 1. The `s` flag is parsed in strings `/s` but not when passed as separate flags in the Regex object CTOR. This seems to have been an oversight in my initial implementation. 2. The RegExp.prototype.dotall property was finalized as `dotAll` with a capital A (to join the rest of the inconsistent casing of RegExp properties in the standard where some have camelCase and some do not). My implementation was `dotall`, therefore coming up with undefined property for `.dotAll` because of case sensitivity. This shouldn't be too hard to fix. See #1284 for the initial implementation. Funny how this has not been an issue for 3 whole years until it got reported.

Verified manually in scratchpad for proper operation.

Labels Milestones

Missing functionality for RegExp.dotAll #2008