Details

    • Type: New Feature New Feature
    • Status: Resolved Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: JRuby 1.7.0.pre1
    • Fix Version/s: JRuby 1.7.0.pre1
    • Component/s: Core Classes/Modules
    • Labels:
      None
    • Environment:
      OSX Lion, JRuby master
    • Number of attachments :
      0

      Description

      This patch adds support for --profile.html which prints the --profile.graph output but in HTML format.

        Activity

        Show
        Ketan Padegaonkar added a comment - Commits on this branch: https://github.com/ketan/jruby/compare/jruby:master...ketan:html-profiler
        Hide
        Charles Oliver Nutter added a comment -

        Wow, I can't believe we missed this for several months.

        The commit still applies perfectly. My only complaint about it is that the output still includes the header information above it. Perhaps this can be improved to dump just the HTML content? Perhaps that's tricky since the formatters run outside of the per-thread output...

        I have no problem pulling the commit to JRuby master so you can rebase from there more easily.

        Show
        Charles Oliver Nutter added a comment - Wow, I can't believe we missed this for several months. The commit still applies perfectly. My only complaint about it is that the output still includes the header information above it. Perhaps this can be improved to dump just the HTML content? Perhaps that's tricky since the formatters run outside of the per-thread output... I have no problem pulling the commit to JRuby master so you can rebase from there more easily.
        Hide
        Charles Oliver Nutter added a comment -

        Actually, I'm just going to resolve this, since your commit is in and it does work. I'd love to see some improvements here, if you have time and interest, but it's in and it's working!

        Show
        Charles Oliver Nutter added a comment - Actually, I'm just going to resolve this, since your commit is in and it does work. I'd love to see some improvements here, if you have time and interest, but it's in and it's working!

          People

          • Assignee:
            Charles Oliver Nutter
            Reporter:
            Ketan Padegaonkar
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: