WordPress.org
Get WordPress

Make WordPress Core

Opened 18 years ago

Closed 18 years ago

Last modified 18 years ago

#8166 closed enhancement (fixed)

Optimizations for mysql2date()

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

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 18 years ago.
First patch which removes redundant code (DRY), based on r9626
8166.r9626.2.diff​ (2.7 KB ) - added by jacobsantos 18 years ago.
Second patch for optimizating mysql2time() based on r9626

Download all attachments as: .zip

Change History (8)

@jacobsantos
18 years ago

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

@jacobsantos
18 years ago

Second patch for optimizating mysql2time() based on r9626

#1 @jacobsantos
18 years ago

  • Keywords has-patch added

#2 @Speedboxer
18 years ago

Tested the second patch, it works.

#3 @jacobsantos
18 years ago

  • Keywords tested commit added

#4 @westi
18 years ago

Liking the ammount of red in the diff.

Lets throw this in trunk

#5 @westi
18 years ago

  • Resolution → fixed
  • Status new → closed

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

Note: See TracTickets for help on using tickets.