WordPress.org
Get WordPress

Make WordPress Core

Opened 16 years ago

Closed 16 years ago

Last modified 16 years ago

#8166 closed enhancement (fixed)

Optimizations for mysql2date()

Reported by: jacobsantos's profile jacobsantos Owned by:
Milestone: 2.8 Priority: normal
Severity: normal Version:
Component: Optimization Keywords: phpdoc has-patch tested commit
Focuses: Cc:

Description

Optimizations for mysql2date().

  1. Might be premature, but first patch at least removes redundant code.
  2. Second patch needs testing, but might speed up code while simpflying it.

Based on Speedboxer suggestion and discussion with DD32 on IRC.

Attachments (2)

8166.r9626.diff​ (2.2 KB) - added by jacobsantos 16 years ago.
First patch which removes redundant code (DRY), based on r9626
8166.r9626.2.diff​ (2.7 KB) - added by jacobsantos 16 years ago.
Second patch for optimizating mysql2time() based on r9626

Download all attachments as: .zip

Change History (8)

@jacobsantos
16 years ago

First patch which removes redundant code (DRY), based on r9626

@jacobsantos
16 years ago

Second patch for optimizating mysql2time() based on r9626

#1 @jacobsantos
16 years ago

  • Keywords has-patch added

#2 @Speedboxer
16 years ago

Tested the second patch, it works.

#3 @jacobsantos
16 years ago

  • Keywords tested commit added

#4 @westi
16 years ago

Liking the ammount of red in the diff.

Lets throw this in trunk

#5 @westi
16 years ago

  • Resolution set to fixed
  • Status changed from new to closed

(In [10408]) Reduce code in mysql2date. Fixes #8166 props jacobsantos

Note: See TracTickets for help on using tickets.