This is the mail archive of the systemtap@sourceware.org mailing list for the systemtap 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 doc/langref.tex] Add error probe information


On 09/04/2009 09:47 AM, Robb Romans wrote:
> On Thu, 3 Sep 2009, Josh Stone wrote:
> 
>> On 09/03/2009 03:18 PM, Robb Romans wrote:
>>> +\subsubsection{error}
>>> +\index{error} The \emph{error} probe point is similar to the end
>>> +probe, except the probe handler runs when the session ends if an error
>>> +occurred.  In this case, an \texttt{end} probe is skipped, but each
>>> +\texttt{error} probe is still attempted.  You can use an
>>> +\texttt{error} probe to clean up or perform a final action on script
>>> +termination. You can also numerically parametrize an \texttt{error}
>>> +probe to set a sequence.
>>
>> The meaning of the sequence isn't really clear until its described in
>> the next subsection for begin and end probes.  Perhaps the error
>> sequence should be moved into that discussion?
> 
> It's similar to an end probe, which is why I put it immediately after that 
> probe. If you think it's clearer, I'll move it below "begin and end probe 
> sequence".

I only meant to merge the sequence bit with the other sequence info.
Right now it goes:

- begin
- end
- begin and end sequence

Your addition makes it:

- begin
- end
- error (mentioning but not explaining sequence)
- begin and end sequence

I'm just suggesting to make it more uniform:

- begin
- end
- error
- begin/end/error sequence

Josh


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