* [GIT PULL] Cosmetic fixes for chapters 4, 5 and 6
@ 2016-06-18 8:43 Akira Yokosawa
2016-06-20 17:32 ` Paul E. McKenney
0 siblings, 1 reply; 4+ messages in thread
From: Akira Yokosawa @ 2016-06-18 8:43 UTC (permalink / raw)
To: paulmck; +Cc: perfbook, Akira Yokosawa
The following changes since commit 17ba9e5472a7eefa5e7af13f708c1df4f772e11b:
cpu: Fix numbers in Performance of Mechanisms tables (2016-06-17 20:54:35 -0700)
are available in the git repository at:
https://github.com/akiyks/perfbook.git cosmetic-fixes
for you to fetch changes up to 0c2b722d652c4da3b3ba7c0521e50f0f2df57d59:
count: Typo fix ("a Intel" -> "an Intel") (2016-06-18 17:20:35 +0900)
----------------------------------------------------------------
Akira Yokosawa (9):
toolsoftrade: Centering figures of verbatim code examples
toolsoftrade: Adjust position of Figure 4.5
toolsoftrade: Use \path command for C source file name
count: Use \path command for C source file name
count: Centering figures of verbatim code examples
count: Tweak position of Figure 5.1
SMPdesign: Use \path command for path names
SMPdesign: Centering figures of verbatim code examples
count: Typo fix ("a Intel" -> "an Intel")
SMPdesign/SMPdesign.tex | 70 ++++++-----
SMPdesign/beyond.tex | 42 ++++---
SMPdesign/partexercises.tex | 32 +++--
count/count.tex | 270 ++++++++++++++++++++++++------------------
perfbook.tex | 1 +
toolsoftrade/toolsoftrade.tex | 98 ++++++++-------
6 files changed, 302 insertions(+), 211 deletions(-)
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [GIT PULL] Cosmetic fixes for chapters 4, 5 and 6
2016-06-18 8:43 [GIT PULL] Cosmetic fixes for chapters 4, 5 and 6 Akira Yokosawa
@ 2016-06-20 17:32 ` Paul E. McKenney
2016-06-21 15:31 ` Akira Yokosawa
0 siblings, 1 reply; 4+ messages in thread
From: Paul E. McKenney @ 2016-06-20 17:32 UTC (permalink / raw)
To: Akira Yokosawa; +Cc: perfbook
On Sat, Jun 18, 2016 at 05:43:23PM +0900, Akira Yokosawa wrote:
> The following changes since commit 17ba9e5472a7eefa5e7af13f708c1df4f772e11b:
>
> cpu: Fix numbers in Performance of Mechanisms tables (2016-06-17 20:54:35 -0700)
>
> are available in the git repository at:
>
> https://github.com/akiyks/perfbook.git cosmetic-fixes
>
> for you to fetch changes up to 0c2b722d652c4da3b3ba7c0521e50f0f2df57d59:
>
> count: Typo fix ("a Intel" -> "an Intel") (2016-06-18 17:20:35 +0900)
>
> ----------------------------------------------------------------
> Akira Yokosawa (9):
> toolsoftrade: Centering figures of verbatim code examples
> toolsoftrade: Adjust position of Figure 4.5
> toolsoftrade: Use \path command for C source file name
> count: Use \path command for C source file name
> count: Centering figures of verbatim code examples
> count: Tweak position of Figure 5.1
> SMPdesign: Use \path command for path names
> SMPdesign: Centering figures of verbatim code examples
> count: Typo fix ("a Intel" -> "an Intel")
>
> SMPdesign/SMPdesign.tex | 70 ++++++-----
> SMPdesign/beyond.tex | 42 ++++---
> SMPdesign/partexercises.tex | 32 +++--
> count/count.tex | 270 ++++++++++++++++++++++++------------------
> perfbook.tex | 1 +
> toolsoftrade/toolsoftrade.tex | 98 ++++++++-------
> 6 files changed, 302 insertions(+), 211 deletions(-)
Thank you, Akira!
I queued this and pushed it, but I am a bit dissatisfied with the position
of Figure 5.1, stuck between the section heading and the text. I agree
that its original position tucked underneath the long footnote 1 is also
not so good.
I queued and pushed the patch below, which puts it above the section
heading, but please feel free to further improve it.
Thanx, Paul
------------------------------------------------------------------------
commit 15ddcdd3ae6bdc088f7c49f78edf62fff5090cfd
Author: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
Date: Mon Jun 20 10:30:49 2016 -0700
Further tweak position of "Just Count!" figure
Signed-off-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
diff --git a/count/count.tex b/count/count.tex
index d0704536c91c..f4b63687892e 100644
--- a/count/count.tex
+++ b/count/count.tex
@@ -180,7 +180,7 @@ are more appropriate for advanced students.
11 }
\end{verbbox}
}
-\begin{figure}[hp]
+\begin{figure}[tp]
\centering
\theverbbox
\caption{Just Count!}
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [GIT PULL] Cosmetic fixes for chapters 4, 5 and 6
2016-06-20 17:32 ` Paul E. McKenney
@ 2016-06-21 15:31 ` Akira Yokosawa
2016-06-21 16:03 ` Paul E. McKenney
0 siblings, 1 reply; 4+ messages in thread
From: Akira Yokosawa @ 2016-06-21 15:31 UTC (permalink / raw)
To: paulmck; +Cc: perfbook, Akira Yokosawa
On 2016/06/20 10:32:09 -0700, Paul E. McKenney wrote:
> On Sat, Jun 18, 2016 at 05:43:23PM +0900, Akira Yokosawa wrote:
>> The following changes since commit 17ba9e5472a7eefa5e7af13f708c1df4f772e11b:
>>
>> cpu: Fix numbers in Performance of Mechanisms tables (2016-06-17 20:54:35 -0700)
>>
>> are available in the git repository at:
>>
>> https://github.com/akiyks/perfbook.git cosmetic-fixes
>>
>> for you to fetch changes up to 0c2b722d652c4da3b3ba7c0521e50f0f2df57d59:
>>
>> count: Typo fix ("a Intel" -> "an Intel") (2016-06-18 17:20:35 +0900)
>>
>> ----------------------------------------------------------------
>> Akira Yokosawa (9):
>> toolsoftrade: Centering figures of verbatim code examples
>> toolsoftrade: Adjust position of Figure 4.5
>> toolsoftrade: Use \path command for C source file name
>> count: Use \path command for C source file name
>> count: Centering figures of verbatim code examples
>> count: Tweak position of Figure 5.1
>> SMPdesign: Use \path command for path names
>> SMPdesign: Centering figures of verbatim code examples
>> count: Typo fix ("a Intel" -> "an Intel")
>>
>> SMPdesign/SMPdesign.tex | 70 ++++++-----
>> SMPdesign/beyond.tex | 42 ++++---
>> SMPdesign/partexercises.tex | 32 +++--
>> count/count.tex | 270 ++++++++++++++++++++++++------------------
>> perfbook.tex | 1 +
>> toolsoftrade/toolsoftrade.tex | 98 ++++++++-------
>> 6 files changed, 302 insertions(+), 211 deletions(-)
>
> Thank you, Akira!
>
> I queued this and pushed it, but I am a bit dissatisfied with the position
> of Figure 5.1, stuck between the section heading and the text. I agree
> that its original position tucked underneath the long footnote 1 is also
> not so good.
>
> I queued and pushed the patch below, which puts it above the section
> heading, but please feel free to further improve it.
Well, as I said in the commit message, it was a compromise.
I compared [hp] and [tp], and picked [hp], because it looks not so good
to have the figure just above the section heading.
I think the best position of the figure in this case is just above the
footnote.
Today, I googled "latex floating figure underneath footnote", and found
that by using 'footmisc' package, it is possible to place a floating figure/
table just above a footnote.
I'm going to test it and if it works as I expect, I'll send a pull request.
Thanks, Akira
>
> Thanx, Paul
>
> ------------------------------------------------------------------------
>
> commit 15ddcdd3ae6bdc088f7c49f78edf62fff5090cfd
> Author: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
> Date: Mon Jun 20 10:30:49 2016 -0700
>
> Further tweak position of "Just Count!" figure
>
> Signed-off-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
>
> diff --git a/count/count.tex b/count/count.tex
> index d0704536c91c..f4b63687892e 100644
> --- a/count/count.tex
> +++ b/count/count.tex
> @@ -180,7 +180,7 @@ are more appropriate for advanced students.
> 11 }
> \end{verbbox}
> }
> -\begin{figure}[hp]
> +\begin{figure}[tp]
> \centering
> \theverbbox
> \caption{Just Count!}
>
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [GIT PULL] Cosmetic fixes for chapters 4, 5 and 6
2016-06-21 15:31 ` Akira Yokosawa
@ 2016-06-21 16:03 ` Paul E. McKenney
0 siblings, 0 replies; 4+ messages in thread
From: Paul E. McKenney @ 2016-06-21 16:03 UTC (permalink / raw)
To: Akira Yokosawa; +Cc: perfbook
On Wed, Jun 22, 2016 at 12:31:26AM +0900, Akira Yokosawa wrote:
> On 2016/06/20 10:32:09 -0700, Paul E. McKenney wrote:
> >On Sat, Jun 18, 2016 at 05:43:23PM +0900, Akira Yokosawa wrote:
> >>The following changes since commit 17ba9e5472a7eefa5e7af13f708c1df4f772e11b:
> >>
> >> cpu: Fix numbers in Performance of Mechanisms tables (2016-06-17 20:54:35 -0700)
> >>
> >>are available in the git repository at:
> >>
> >> https://github.com/akiyks/perfbook.git cosmetic-fixes
> >>
> >>for you to fetch changes up to 0c2b722d652c4da3b3ba7c0521e50f0f2df57d59:
> >>
> >> count: Typo fix ("a Intel" -> "an Intel") (2016-06-18 17:20:35 +0900)
> >>
> >>----------------------------------------------------------------
> >>Akira Yokosawa (9):
> >> toolsoftrade: Centering figures of verbatim code examples
> >> toolsoftrade: Adjust position of Figure 4.5
> >> toolsoftrade: Use \path command for C source file name
> >> count: Use \path command for C source file name
> >> count: Centering figures of verbatim code examples
> >> count: Tweak position of Figure 5.1
> >> SMPdesign: Use \path command for path names
> >> SMPdesign: Centering figures of verbatim code examples
> >> count: Typo fix ("a Intel" -> "an Intel")
> >>
> >> SMPdesign/SMPdesign.tex | 70 ++++++-----
> >> SMPdesign/beyond.tex | 42 ++++---
> >> SMPdesign/partexercises.tex | 32 +++--
> >> count/count.tex | 270 ++++++++++++++++++++++++------------------
> >> perfbook.tex | 1 +
> >> toolsoftrade/toolsoftrade.tex | 98 ++++++++-------
> >> 6 files changed, 302 insertions(+), 211 deletions(-)
> >
> >Thank you, Akira!
> >
> >I queued this and pushed it, but I am a bit dissatisfied with the position
> >of Figure 5.1, stuck between the section heading and the text. I agree
> >that its original position tucked underneath the long footnote 1 is also
> >not so good.
> >
> >I queued and pushed the patch below, which puts it above the section
> >heading, but please feel free to further improve it.
>
> Well, as I said in the commit message, it was a compromise.
> I compared [hp] and [tp], and picked [hp], because it looks not so good
> to have the figure just above the section heading.
> I think the best position of the figure in this case is just above the
> footnote.
>
> Today, I googled "latex floating figure underneath footnote", and found
> that by using 'footmisc' package, it is possible to place a floating figure/
> table just above a footnote.
> I'm going to test it and if it works as I expect, I'll send a pull request.
Sounds good, looking forward to seeing the results.
Thanx, Paul
> >------------------------------------------------------------------------
> >
> >commit 15ddcdd3ae6bdc088f7c49f78edf62fff5090cfd
> >Author: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
> >Date: Mon Jun 20 10:30:49 2016 -0700
> >
> > Further tweak position of "Just Count!" figure
> >
> > Signed-off-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
> >
> >diff --git a/count/count.tex b/count/count.tex
> >index d0704536c91c..f4b63687892e 100644
> >--- a/count/count.tex
> >+++ b/count/count.tex
> >@@ -180,7 +180,7 @@ are more appropriate for advanced students.
> > 11 }
> > \end{verbbox}
> > }
> >-\begin{figure}[hp]
> >+\begin{figure}[tp]
> > \centering
> > \theverbbox
> > \caption{Just Count!}
> >
> >
>
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2016-06-21 19:01 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-06-18 8:43 [GIT PULL] Cosmetic fixes for chapters 4, 5 and 6 Akira Yokosawa
2016-06-20 17:32 ` Paul E. McKenney
2016-06-21 15:31 ` Akira Yokosawa
2016-06-21 16:03 ` Paul E. McKenney
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.