Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Show cumulative compile time #27

Closed
wants to merge 6 commits into from
Closed

Show cumulative compile time #27

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
da630d6
Only build regexp once
eaigner May 23, 2016
1344f27
Clarify cancel expression
eaigner May 23, 2016
0973de1
Remove unnecessary declarations
eaigner May 23, 2016
d38beb3
Accumulate measure time for identical result lines
eaigner May 23, 2016
cab9f22
Remove results cap
eaigner May 23, 2016
ba83ded
Group results by source file line
eaigner May 23, 2016
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
1 change: 1 addition & 0 deletions .gitignore
View file
Open in desktop
Expand Up @@ -61,3 +61,4 @@ Carthage/Build

fastlane/report.xml
fastlane/screenshots
.DS_Store
10 changes: 7 additions & 3 deletions BuildTimeAnalyzer/CMCompileMeasure.swift
View file
Open in desktop
Expand Up @@ -14,11 +14,15 @@ struct CMCompileMeasure {
var path: String
var code: String
var filename: String

private var locationArray: [Int]


var fileAndLine: String {
return "\(filename):\(locationArray[0])"
}

var fileInfo: String {
return "\(filename):\(locationArray[0]):\(locationArray[1])"
return "\(fileAndLine):\(locationArray[1])"
}

var location: Int {
Expand Down
73 changes: 43 additions & 30 deletions BuildTimeAnalyzer/CMLogProcessor.swift
View file
Open in desktop
Expand Up @@ -18,6 +18,8 @@ protocol CMLogProcessorProtocol: class {
func processingDidFinish()
}

private let processRx = try! NSRegularExpression(pattern: "^\\d*\\.?\\dms\\t/", options: [])

extension CMLogProcessorProtocol {
func process(productName: String, buildCompletionDate: NSDate?, updateHandler: CMUpdateClosure?) {
workspace = CMXcodeWorkSpace(productName: productName, buildCompletionDate: buildCompletionDate)
Expand All @@ -37,61 +39,58 @@ extension CMLogProcessorProtocol {
// MARK: Private methods

private func process(text text: String) {
let locationPattern = "^\\d*\\.?\\dms\\t/"
let matchingOption = NSMatchingOptions(rawValue: 0)
let compareOptions = NSStringCompareOptions(rawValue: 0)
let regexOptions = NSRegularExpressionOptions(rawValue: 0)
let characterSet = NSCharacterSet(charactersInString:"\r\"")

let regex = try! NSRegularExpression(pattern: locationPattern, options: regexOptions)

var remainingRange = text.startIndex..<text.endIndex

var rawMeasures: [String:Double] = [:]

unprocessedResult.removeAll()
processingDidStart()

while let nextRange = text.rangeOfCharacterFromSet(characterSet, options: compareOptions, range: remainingRange) {
while let nextRange = text.rangeOfCharacterFromSet(characterSet, options: [], range: remainingRange) {
let currentRange = remainingRange.startIndex..<nextRange.endIndex
let text = text.substringWithRange(currentRange)

defer { remainingRange = nextRange.endIndex..<remainingRange.endIndex }
defer {
remainingRange = nextRange.endIndex..<remainingRange.endIndex
}

let range = NSMakeRange(0, text.characters.count)
guard let match = regex.firstMatchInString(text, options: matchingOption, range: range) else { continue }
guard let match = processRx.firstMatchInString(text, options: [], range: range) else { continue }

let timeString = text.substringToIndex(text.startIndex.advancedBy(match.range.length - 4))
if let time = Double(timeString) {
let value = text.substringFromIndex(text.startIndex.advancedBy(match.range.length - 1))
unprocessedResult.append(CMRawMeasure(time: time, text: value))

let cumulativeTime = rawMeasures[value] ?? 0
rawMeasures[value] = cumulativeTime + time
}
if shouldCancel {
break
}
}

for (k, v) in rawMeasures {
// Only show files whose compile time is above 10ms
if v > 10 {
unprocessedResult.append(CMRawMeasure(time: v, text: k))
}
guard !shouldCancel else { break }
}

processingDidFinish()
}

private func updateResults(didComplete: Bool) {
let cappedResult = capEntries(unprocessedResult)
updateHandler?(result: processResult(cappedResult), didComplete: didComplete)

var results = processResult(unprocessedResult)
results = groupResultsBySourceLine(results)
results.sortInPlace{ $0.time > $1.time }

updateHandler?(result: results, didComplete: didComplete)
if didComplete {
unprocessedResult.removeAll()
}
}

private func capEntries(entries: [CMRawMeasure]) -> [CMRawMeasure] {
let limit = 20

let distinct = Array(Set(entries))
var sorted = distinct.sort{ $0.time > $1.time }
if sorted.count > limit {
sorted = Array(sorted[0..<limit])
}
return sorted
}


private func processResult(unprocessedResult: [CMRawMeasure]) -> [CMCompileMeasure] {
let unprocessedResult = capEntries(unprocessedResult)

var result: [CMCompileMeasure] = []
for entry in unprocessedResult {
let code = entry.text.characters.split("\t").map(String.init)
Expand All @@ -101,6 +100,20 @@ extension CMLogProcessorProtocol {
}
return result
}

private func groupResultsBySourceLine(results: [CMCompileMeasure]) -> [CMCompileMeasure] {
var grouped: [String:CMCompileMeasure] = [:]
for result in results {
let key = result.fileAndLine
if var measure = grouped[key] {
measure.time += result.time
grouped[key] = measure
} else {
grouped[key] = result
}
}
return Array(grouped.values)
}

private func trimPrefixes(code: String) -> String {
var code = code
Expand Down