This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB 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] arm reversible : <phase_2_complete>

From: Tom Tromey <tromey at redhat dot com>
To: oza Pawandeep <oza dot pawandeep at gmail dot com>
Cc: Yao Qi <yao at codesourcery dot com>, Petr HluzÃn <petr dot hluzin at gmail dot com>, paawan oza <paawan1982 at yahoo dot com>, "gdb-patches\ at sourceware dot org" <gdb-patches at sourceware dot org>, chandra krishnappa <chandra_roadking at yahoo dot com>
Date: Tue, 20 Dec 2011 12:05:09 -0700
Subject: Re: [PATCH] arm reversible : <phase_2_complete>
References: <998639.46560.qm@web112516.mail.gq1.yahoo.com> <m3vcqieb48.fsf@fleche.redhat.com> <CAK1A=4wz=2uU71yFz39EdOMvrLEyR+dX9TEogJ6vRR2Pu4pDNw@mail.gmail.com> <CAK1A=4zrPC2WnP8_F+pF9djgxDsZJMu6hbVMVVZXvC-VoqOHjQ@mail.gmail.com> <m3hb1i95xn.fsf@fleche.redhat.com> <CAK1A=4xgZiBR=iWgo_AbUw2amVZstqyQ3L-M2wm6DWndSh16Cw@mail.gmail.com> <CAK1A=4xBYsQ129h7GmF+KyL_0oz09+Kr0a4A+X-ir41-=DVSug@mail.gmail.com> <CAC=yr6CVW+5gOXCpo_jC+ESxrFWUqb=DakDvbT5URsd+F8Y_ZA@mail.gmail.com> <CAK1A=4zVVw3Y2pVTQ4EZmwcLtQob-QUajTyaSWEk4YQM_9b=MA@mail.gmail.com> <CAK1A=4wQis+_YW=oWBtreGZsXDZfK=FbTQv5uHsb=1Kp15=0-w@mail.gmail.com> <4EDAD0EF.20405@codesourcery.com> <CAK1A=4yLcpXq02hTt7xHEkSj1Z_8jc5Ew_3DY1GzqzWu_u=8gg@mail.gmail.com> <CAK1A=4wT8UrgdaOPDDggvdHBMnEh9b1mNLnOko9s51EV1zTXEQ@mail.gmail.com> <CAK1A=4xXnD0f05zXhFDC_f_Z0kroR46r=L5HLnSeTJK7yU9oUg@mail.gmail.com> <4EDC0629.5060208@codesourcery.com> <CAK1A=4x7eSEmAd8j8OCxs4AWewk9rVqCrZZrJV+_wb2-P8Qkpg@mail.gmail.com> <4EDC7CF2.1090505@codesourcery.com> <CAK1A=4zgo5qppwCF5C1SOk=Mn4dn56-sCTMXKR63YwzOsWyi0g@mail.gmail.com>

>>>>> "Oza" == oza Pawandeep <oza.pawandeep@gmail.com> writes:

Oza> here is the latest patch with Changelog comment fixed.

One last little nit...

Oza> +      gdb_assert (0);

Please change this to gdb_assert_not_reached.

This is ok with this change.

Could you write a NEWS entry for this change?

Tom

Follow-Ups:
- Re: [PATCH] arm reversible : <phase_2_complete>
  - From: oza Pawandeep

References:
- Re: [PATCH] arm reversible : <phase_2_complete>
  - From: Tom Tromey
- Re: [PATCH] arm reversible : <phase_2_complete>
  - From: oza Pawandeep
- Re: [PATCH] arm reversible : <phase_2_complete>
  - From: oza Pawandeep
- Re: [PATCH] arm reversible : <phase_2_complete>
  - From: Petr HluzÃn
- Re: [PATCH] arm reversible : <phase_2_complete>
  - From: oza Pawandeep
- Re: [PATCH] arm reversible : <phase_2_complete>
  - From: oza Pawandeep
- Re: [PATCH] arm reversible : <phase_2_complete>
  - From: Yao Qi
- Re: [PATCH] arm reversible : <phase_2_complete>
  - From: oza Pawandeep
- Re: [PATCH] arm reversible : <phase_2_complete>
  - From: oza Pawandeep
- Re: [PATCH] arm reversible : <phase_2_complete>
  - From: oza Pawandeep
- Re: [PATCH] arm reversible : <phase_2_complete>
  - From: Yao Qi
- Re: [PATCH] arm reversible : <phase_2_complete>
  - From: oza Pawandeep
- Re: [PATCH] arm reversible : <phase_2_complete>
  - From: Yao Qi
- Re: [PATCH] arm reversible : <phase_2_complete>
  - From: oza Pawandeep

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