- User Since
- Dec 25 2017, 5:51 AM (72 w, 6 d)
Fri, May 17
PS: I have no idea why the macro note is purple, but because the entire HTML printing is broken I hope you saw nothing.
Thanks you! I hope this feature will be useful to see what is behind the scenes and create better reports.
- Add extra state transition indices to every pop-up note.
@NoQ could you share your final thoughts here, please? I would like to move forward and never look back.
Thanks for the accept! Sorry for the extra noise but it came out it is tricky to create valid JSON for each sub-print and we do not need it for now. I wanted to create the most compact and necessary JSON without touching the JsonSupport.h.
- Better header handling.
- Less JSON.
Thu, May 16
I do not like to have a variable storing/mention one stuff named in plural. It is your decision as it is just my personal feeling.
Wed, May 15
PrettyStackTraceLocationContext.h definitely needs the default print() method (may other users too). Even it is a little bit silly but I think we have to keep them and create printJSON(). What do you think?
- More JSONify.
Tue, May 14
What about "the most derived class" or "a superclass" instead of "the superclass"? (https://en.cppreference.com/w/cpp/language/derived_class)
May the sentence is a little bit too long. It would be cool to say "by A" or something more simple and precise.
Mon, May 13
PS: I mean we do not write it out and I am not sure whether it is the first/last of the ClusterBindings or what.
Thu, May 9
Abandoned like its parent revision.
I see, now I get it. Thanks for the idea! Sadly Graphviz has a very limited HTML usage so this tool has to do a lot more. I like that idea to have machine code inside the graph so I can parse it and modify it whatever I like without any extra overhead of very complex LLVM file-handling/command execution. I will mention the source of these ideas in the actual patch.
In addition: If I am right you would use the CTU-level more complex JSON format with tons of additional stuff compared to the current raw ugly DOT dump. So forget about the current DOT as it useless compared to the JSON and convert the JSON into DOT so to SVG and HTML?
Would you introduce printJSON() instead of printDiff() in every graph-printer method then just dumping it out into a file? I do not see any generic way.
Wed, May 8
Thanks for the ideas! I am looking forward for that JSON formatting as it sounds good.
This patch is the sample for every other printDiff() functions, so let us brainstorming. I think it is useless to write out if there is no new information, also it comes with an ugly code. What do you guys think?
PS: I do not like that fraction stuff as it is difficult to read, ugly and limited.
I have not got better idea to just index over each sub-state. Example at line 1052:
- Sub-state indexing.
Sat, May 4
I think the entire LocationContext stuff is a huge design issue, and you used its functionality without any hack. If you would rename the getStackFrame to getNextStackFrame or something, it is clear it is not a getter and traversing every frame until the top-frame.
Wed, May 1
PS: That offset-by-one index + arrow confuse me more, so that I am not sure about that improvement being a useful one.
Thanks for the review!
index → index+1
Mon, Apr 29
When I see ->* I go mad, but we should handle unknown stuff, cool patch.
Sun, Apr 28
Great patch! There is only a design problem:
You have negated every isSmall() condition looks like you could write isLarge() instead but there is no connection between these functions.
Could you rename or redesign them? The following would be cool: isLarge() iff !isSmalll and isSmall() iff !isLarge().
Sat, Apr 27
The indexing has a problem: zero index would be possible and usually you would match the gray PathDiagnostic bubbles with these notes.
Fix multiple reports at the same range:
- Plain text became a table.
- Styling is the same as the bubbles.
- mrange (whatever it means) - the grey highlight cannot overlap pop-up ranges.
Wed, Apr 24
Feature request test is in D61060.
- Revert clang-format.
Mon, Apr 22
@Szelethus, I will revert it.
Apr 17 2019
Apr 15 2019
I think this functionality is unused because you would split the file into six to reduce the overhead/scroll and that is it.
Apr 14 2019
Thanks you @NoQ for the great idea!
Treat the reports as it is.
I am okay with the grammatical sense, just we are programmers.
Address that only "variable" which is an enum in D60671.
This error found in D59155.
- Removed member variable.
- Introducing PathDiagnosticPopUpPiece(D60670)
Apr 4 2019
It is very good to try one improvement in another similar function.
Mar 29 2019
Mar 28 2019
Mar 27 2019
I like the idea to put connecting stuff together in the same file to create more understandable code. LGTM.
There is no connection with my reverse-engineering reverted patch: https://reviews.llvm.org/D57410?id=184992 ? It evaluates the left and the right side and may it could help.
Mar 26 2019
Cool! I have found this message-semantic idea useful in Unity where every GameObject could talk with each other in a very generic way (https://docs.unity3d.com/ScriptReference/GameObject.SendMessage.html).
I am looking forward for more!