Make label mandatory for listing environment

cdlm · Feb 29, 2016 · 37b78b4 · 37b78b4
1 parent ed9f653
commit 37b78b4
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/sbabook.cls b/sbabook.cls
@@ -311,11 +311,12 @@
   /tcb/code = {\appto\tcb@new@colopt{,before title={\tcb@blend@beforetitle{\lstlistingname~\thefigure}}}}
   % crefname={listing}{listings},
   % Crefname={Listing}{Listings}
-]{listing}[3][]{
+]{listing}[4][]{
   sba={#2},
-  title={#3},
+  label=#3,
+  title={#4},
   bottomtitle = 1ex,
-  list entry = {\protect\numberline{\thefigure}{\ignorespaces #3}},
+  list entry = {\protect\numberline{\thefigure}{\ignorespaces #4}},
   #1
 }
 

diff --git a/sbabook.tex b/sbabook.tex
@@ -271,7 +271,7 @@ \section{Source code and listings}
 Similarly, use the \code{list text} option from \code{tcolorbox} if the caption
 should be worded differently in the table of contents.
 
-\begin{listing}[label=lst:fooNewWith]{Smalltalk}
+\begin{listing}{Smalltalk}{lst:fooNewWith}
 {A factory method in class \code{Foo}}
 Foo class>>with: parameter
   ^ self new