This is the mail archive of the cygwin-patches mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] better stackdumps


On Tue, 18 Mar 2008, Brian Dessent wrote:

> Brian Dessent wrote:
>
> > Of course the labeling works for any module/dll, not just cygwin1.dll,
> > but I didn't have a more elaborate testcase to demonstrate.
>
> Forgot to mention...
>
> The symbols are just tacked on on the right hand side there for now.  I
> wasn't really sure how to handle that.  I didn't want to remove display
> of the actual EIP for each frame as that could be removing useful info,
> but I wasn't quite sure where to put everything or how to align it... so
> as it is now it wraps wider than 80 chars which is probably pretty ugly
> on a default size terminal.

Would it make sense to force a newline before the function name and to
display it with a small indent?  That way people who want the old-style
stackdump could just feed the new one into "grep -v '^  '" or something...
	Igor
-- 
				http://cs.nyu.edu/~pechtcha/
      |\      _,,,---,,_	    pechtcha@cs.nyu.edu | igor@watson.ibm.com
ZZZzz /,`.-'`'    -.  ;-;;,_		Igor Peshansky, Ph.D. (name changed!)
     |,4-  ) )-,_. ,\ (  `'-'		old name: Igor Pechtchanski
    '---''(_/--'  `-'\_) fL	a.k.a JaguaR-R-R-r-r-r-.-.-.  Meow!

"That which is hateful to you, do not do to your neighbor.  That is the whole
Torah; the rest is commentary.  Go and study it." -- Rabbi Hillel


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]