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.

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

Add a "-- CUT HERE --" marker to exception output #7369

Merged
merged 1 commit into from
Jun 11, 2020

Conversation

earlephilhower
Copy link
Collaborator

Users aren't reporting/seeing the actual PC of failures because they
are only copying the >>stack<< into the decoder and not the Exception
line. Add an ALL CAPS "CUT HERE FOR EXCEPTION DECODER" line to the
output to make it blindingly obvious to include the full crashdump
both in bug reports as well as exception decodes.

Users aren't reporting/seeing the actual PC of failures because they
are only copying the >>stack<< into the decoder and not the Exception
line.  Add an ALL CAPS "CUT HERE FOR EXCEPTION DECODER" line to the
output to make it blindingly obvious to include the full crashdump
both in bug reports as well as exception decodes.
@earlephilhower earlephilhower added this to the 2.7.2 milestone Jun 11, 2020
Copy link
Collaborator

@d-a-v d-a-v left a comment

Choose a reason for hiding this comment

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

Users like to be guided, this is appreciated

@earlephilhower
Copy link
Collaborator Author

An example of the modified output:

--------------- CUT HERE FOR EXCEPTION DECODER ---------------

Exception (29):
epc1=0x4020104f epc2=0x00000000 epc3=0x00000000 excvaddr=0x00000000 depc=0x00000000

>>>stack>>>

ctx: cont
sp: 3ffffe00 end: 3fffffc0 offset: 0190
3fffff90:  3fffdad0 00000000 3ffee304 40201049  
3fffffa0:  feefeffe feefeffe 3ffee32c 40201970  
3fffffb0:  feefeffe feefeffe 3ffe84e0 40100b85  
<<<stack<<<

--------------- CUT HERE FOR EXCEPTION DECODER ---------------

The decoder works w/ or w/o the --- lines (it ignores what it can't understand.).

@earlephilhower earlephilhower merged commit a70e834 into esp8266:master Jun 11, 2020
@earlephilhower earlephilhower deleted the cuthere branch June 11, 2020 22:24
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.

None yet

2 participants