Re: Start thread in constructor

In the book "Java Threads" 3rd ed. by Oaks and Wong, we find the
following code example (p. 164):

public class FibonacciProducer implements Runable {
  // <snip> declaration of queue and thr as members
  public FibonacciProducer (BlockingQueue<Integer> q){
    queue = q;
    thr = new Thread(this);
    thr.start();
  }

  public void run(){
    // snip
  }
}

I read somewhere that it is bad practice to start a thread in the
constructor because this may publish the this pointer before the
constructor has returned. Is this not true in this case? Or should we
better code the above with a factory?