Ticket #1483 (closed patch: fixed)

Opened 5 years ago

Last modified 5 years ago

optimization for mixed_cs_index

Reported by: mls Owned by:
Priority: normal Milestone:
Component: core Version: 2.1.0
Severity: medium Keywords:
Cc: Language:
Patch status: rejected Platform:

Description

The attached patch speeds up mixed_cs_index by calculating the codepoint of the first character of the needle string only once instead of for doing so for each character of the haystack string.

Attachments

mixed_cs_index.diff Download (2.2 KB) - added by mls 5 years ago.

Change History

Changed 5 years ago by mls

Changed 5 years ago by jkeenan

Are there any string experts out there who could evaluate this patch?

Thank you very much.

kid51

Changed 5 years ago by NotFound

  • status changed from new to closed
  • resolution set to fixed
  • patch set to rejected

The function has changed a lot since, and no longer have the issue addressed by the patch. Rejecting.

Note: See TracTickets for help on using tickets.