public inbox for linux-scsi@vger.kernel.org
 help / color / mirror / Atom feed
From: Doug Maxey <dwm@enoyolf.org>
To: Mike Christie <michaelc@cs.wisc.edu>
Cc: Ravi Anand <ravi.anand@qlogic.com>,
	David Somayajulu <david.somayajulu@qlogic.com>,
	open-iscsi@googlegroups.com, linux-scsi@vger.kernel.org
Subject: Re: [PATCH 3/3] qla4xxx: iospace_config move defn
Date: Wed, 09 Aug 2006 15:38:42 -0500	[thread overview]
Message-ID: <20060809203842.489939@bebe.enoyolf.org> (raw)
In-Reply-To: <44DA236C.7050900@cs.wisc.edu>


On Wed, 09 Aug 2006 14:03:24 EDT, Mike Christie wrote:
> Mike Christie wrote:
> > 
> > Is my mailer messing up the tabs or are you (maybe the original code was
> > messed up and did not get fixed in the move)?
> > 

Wellll..

Have not been commenting on the whitespace changes.  It appears that 
some frobnication was happening, I have been silently editing leading 
spaces to tabs and stripping trailing whitespace.

Are you seeing tabs changed to spaces?  Or the other way around?  It is
possible that emacs is getting involved some, but I can easily see the
trailing whitespace. (nee '(show-trailing-whitespace t)).

There is also a vi macro to do the same.  My vi experience is too
ancient to comment about the placement in the rc of the following,
but does work for me when I have to fall back to the ubiquitous editor. 
YMMV.

~/.vimrc
...
" Switch syntax highlighting on, when the terminal has colors
" Also switch on highlighting the last used search pattern.
if &t_Co > 2 || has("gui_running")
  syntax on
  set hlsearch
  highlight WhitespaceEOL ctermbg=red guibg=red
  match WhitespaceEOL /\s\+$/
endif
...

> 
> Nevermind, I switch mailers and it looks ok now. Weird, some patches
> looked ok and some did not before.
> 



  reply	other threads:[~2006-08-09 20:45 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-08-09 18:24 [PATCHSET] qla4xxx: more qla4xxx cleanup Doug Maxey
2006-08-09 18:42 ` [PATCH 1/3] qla4xxx: clarify doc for probe_adapter Doug Maxey
2006-08-09 18:42 ` [PATCH 2/3] qla4xxx: really move free_adapter Doug Maxey
2006-08-09 18:42 ` [PATCH 3/3] qla4xxx: iospace_config move defn Doug Maxey
2006-08-09 17:56   ` Mike Christie
2006-08-09 18:03     ` Mike Christie
2006-08-09 20:38       ` Doug Maxey [this message]
2006-08-09 19:23     ` David C Somayajulu
2006-08-09 20:15       ` Andrew Vasquez
2006-08-09 20:21         ` James Bottomley
2006-08-09 22:16           ` David C Somayajulu

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20060809203842.489939@bebe.enoyolf.org \
    --to=dwm@enoyolf.org \
    --cc=david.somayajulu@qlogic.com \
    --cc=linux-scsi@vger.kernel.org \
    --cc=michaelc@cs.wisc.edu \
    --cc=open-iscsi@googlegroups.com \
    --cc=ravi.anand@qlogic.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox