Skip to content

Conversation

@achille-roussel
Copy link
Contributor

@achille-roussel achille-roussel commented May 9, 2023

image

Signed-off-by: Achille Roussel <achille.roussel@gmail.com>
Signed-off-by: Achille Roussel <achille.roussel@gmail.com>
Signed-off-by: Achille Roussel <achille.roussel@gmail.com>
Signed-off-by: Achille Roussel <achille.roussel@gmail.com>
Signed-off-by: Achille Roussel <achille.roussel@gmail.com>
Copy link
Contributor

@pelletier pelletier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Happy to help with the CSS if we ever feel the need to pimp it 😄


func indexTmplExecute(w io.Writer, guest, host []profileEntry) error {
var b bytes.Buffer
b.WriteString(`<html>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there any reason why you didn't want to use html/template for this? Feels like it would be easier to read and maintain. :)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The only reason is I copied the code from net/http/pprof :)

@achille-roussel achille-roussel merged commit caf01a2 into main May 9, 2023
@achille-roussel achille-roussel deleted the host-pprof branch May 9, 2023 19:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants