On Wed, Aug 04, 2010 at 10:51:46AM -0500, Herbert Schulz wrote:
> 
> On Aug 4, 2010, at 10:41 AM, Khaled Hosny wrote:
> 
> > On Tue, Aug 03, 2010 at 05:01:01PM +0200, Ulrike Fischer wrote:
> >> Am Tue, 3 Aug 2010 09:56:45 +0200 schrieb Yves Codet:
> >> 
> >>> Hello.
> >>> 
> >>> I think I found a bug in xltxtra, which is illustrated in the attached  
> >>> file. If xltxtra is loaded, the word which is in italics in the  
> >>> section title is also in italics in the running head, but it should be  
> >>> in roman since the running head is in italics; if only fontspec is  
> >>> loaded, the running head is alright.
> >> 
> >> No the running head is not in italic, it uses \slshape. And this is
> >> the source of the problem: A current fontspec or your xltxtra
> >> redefines \em/\emph so that it switches to upshape only if the
> >> current shape is \itshape. 
> >> 
> >> \listfiles
> >> \documentclass{book}
> >> 
> >> \usepackage{fontspec} %version v2.0b, loads fontspec-patches
> >> 
> >> \begin{document}
> >> {\slshape blub \emph{bla}} %slshape + itshape
> >> 
> >> {\itshape blubb \emph{blabla}} %itshape + upshape
> >> 
> >> \end{document}
> > 
> > Since Will is busy with his thesis ;), I tried to fix this issue. The
> > attached fontspec-patches.sty seems to fix the issue for me (fontspec 2
> > only, of course), could some one else test it?
> > 
> > Regards,
> > Khaled
> 
> Howdy,
> 
> Sigh... no attachment here. :-(

Well, I think I need one of those mail composers that remind you of
missed attachments :)

Regards,
 Khaled

-- 
 Khaled Hosny
 Arabic localiser and member of Arabeyes.org team
 Free font developer
%%
%% This is file `fontspec-patches.sty',
%% generated with the docstrip utility.
%%
%% The original source files were:
%%
%% fontspec.dtx  (with options: `patches')
%% 
%%   _________________________________________
%%   The fontspec package for XeLaTeX/LuaLaTeX
%%   (C) 2004--2010    Will Robertson and Khaled Hosny
%% 
%%   License information appended.
%% 
%% 
\ProvidesPackage{fontspec-patches}
  [2010/08/01 v2.0c Advanced font selection for XeLaTeX/LuaLaTeX]



\ExplSyntaxOn
\RequirePackage{fixltx2e}[2006/03/24]
\DeclareRobustCommand \em {
  \...@nomath\em
  \tl_if_eq:xxTF \...@shape \itdefault \eminnershape
  {
    \tl_if_eq:xxTF \...@shape \sldefault \eminnershape \emshape
  }
}
\DeclareTextFontCommand{\emph}{\em}
\let\emshape\itshape
\let\eminnershape\upshape
\DeclareRobustCommand{\-}{%
  \discretionary{%
    \char\ifnum\hyphenchar\font<\z@
           \...@defaulthyphenchar
         \else
           \hyphenchar\font
         \fi}{}{}}
\def\...@defaulthyphenchar{`\-}
\cs_new:Npn \fontspec_visible_space: {
  \fontspec_glyph_if_exist:NnTF \font {"2423}
    {\char"2423\relax}
    {\fontspec_visible_space_fallback:}
}
\cs_new:Npn \fontspec_visible_space_fallback: {
  {
    \usefont{eu1}{lmtt}...@series}{\f@shape}
    \textvisiblespace
  }
}
\group_begin:
\char_make_active:n{"20}%
\cs_gset:Npn\fontspec_print_visible_spaces:{%
\char_make_active:n{"20}%
\cs_set_eq:NN^^20\fontspec_visible_space:%
}%
\group_end:
\def\verb{
  \relax\ifmmode\hbox\else\leavevmode\null\fi
  \bgroup
    \v...@eol@error \let\...@makeother \dospecials
    \verba...@font\@noligs
    \...@ifstar\@@sve...@verb
}
\def\@@sverb{\fontspec_print_visible_spaces:\...@sverb}
\AtBeginDocument{
  \fontspec_patch_verbatim:
  \fontspec_patch_moreverb:
  \fontspec_patch_fancyvrb:
  \fontspec_patch_listings:
}
\cs_set:Npn \fontspec_patch_verbatim: {
  \...@ifpackageloaded{verbatim}{
    \cs_set:cpn {verbatim*} {
      \begingroup \...@verbatim \fontspec_print_visible_spaces: \verba...@start
    }
  }{
    \cs_set:cpn {verbatim*} {
      \...@verbatim \fontspec_print_visible_spaces: \...@sxverbatim
    }
  }
}
\cs_set:Npn \fontspec_patch_moreverb: {
  \...@ifpackageloaded{moreverb}{
    \cs_set:cpn {listingcont*} {
      \cs_set:Npn \verba...@processline {
        \thelist...@line \global\advance\list...@line\c_one
        \the\verba...@line\par
      }
      \...@verbatim \fontspec_print_visible_spaces: \verba...@start
    }
  }{}
}
\cs_set:Npn \fontspec_patch_fancyvrb: {
  \...@ifpackageloaded{fancyvrb}{
    \cs_set_eq:NN \FancyVerbSpace \fontspec_visible_space:
  }{}
}
\cs_set:Npn \fontspec_patch_listings: {
  \...@ifpackageloaded{listings}{
    \cs_set_eq:NN \...@visiblespace \fontspec_visible_space:
  }{}
}
%% 
%% Copyright 2004--2010 Will Robertson <wsp...@gmail.com>
%% Copyright 2009--2010 Khaled Hosny <khaledho...@eglug.org>
%% 
%% Distributable under the LaTeX Project Public License,
%% version 1.3c or higher (your choice). The latest version of
%% this license is at: http://www.latex-project.org/lppl.txt
%% 
%% This work is "author-maintained" by Will Robertson.
%% 
%% This work consists of this file fontspec.dtx
%%           and the derived files fontspec.sty,
%%                                 fontspec.lua,
%%                                 fontspec.cfg,
%%                                 fontspec-xetex.tex,
%%                                 fontspec-luatex.tex,
%%                             and fontspec.pdf.
%% 
%%
%% End of file `fontspec-patches.sty'.

--------------------------------------------------
Subscriptions, Archive, and List information, etc.:
  http://tug.org/mailman/listinfo/xetex

Reply via email to