Re: [Wireshark-dev] r54005 by wmeier for packet-mq.c and packet-mq-pcf.c

2013-12-18 Thread Bill Meier

On 12/14/2013 5:30 AM, RobiOneKenobi wrote:

Yes,

if it didn't disturb too much people to have such long lines, I will prefer
that you revert this part (hf[] entries reformatting), otherwise I will
follow the majority wishses



I've restored the single line per hf[] entry format in SVN #54005.

Bill



___
Sent via:Wireshark-dev mailing list wireshark-dev@wireshark.org
Archives:http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
mailto:wireshark-dev-requ...@wireshark.org?subject=unsubscribe


Re: [Wireshark-dev] r54005 by wmeier for packet-mq.c and packet-mq-pcf.c

2013-12-18 Thread Bill Meier

On 12/18/2013 11:57 AM, Bill Meier wrote:

On 12/14/2013 5:30 AM, RobiOneKenobi wrote:

Yes,

if it didn't disturb too much people to have such long lines, I will
prefer
that you revert this part (hf[] entries reformatting), otherwise I will
follow the majority wishses



I've restored the single line per hf[] entry format in SVN #54005.




Correction: The restoration was done in SVN #54226.

___
Sent via:Wireshark-dev mailing list wireshark-dev@wireshark.org
Archives:http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
mailto:wireshark-dev-requ...@wireshark.org?subject=unsubscribe


Re: [Wireshark-dev] r54005 by wmeier for packet-mq.c and packet-mq-pcf.c

2013-12-14 Thread RobiOneKenobi
Yes, 

if it didn't disturb too much people to have such long lines, I will prefer
that you revert this part (hf[] entries reformatting), otherwise I will
follow the majority wishses

-Original Message-
From: wireshark-dev-boun...@wireshark.org
[mailto:wireshark-dev-boun...@wireshark.org] On Behalf Of Bill Meier
Sent: Friday, December 13, 2013 7:08 PM
To: Developer support list for Wireshark
Subject: Re: [Wireshark-dev] r54005 by wmeier for packet-mq.c and
packet-mq-pcf.c

On 12/13/2013 12:21 PM, RobiOneKenobi wrote:
 Reformat hf[] entries ?

 Why, now all seems no more aligned for me, and your reformat also
 left spaces between text and comma.

You're right; I was sloppy about leaving spaces between the text and the 
comma.   :)


 I do not agree with such reformat, as it seems less readable as the
 one I've put in.


I made the change because, personally, I find the quite long lines quite
difficult to read when they when they exceed the width of my screen
(which I expect they will do on screens used by many).

 I like to have some parts aligned to an ease of use for to have the
 same length of displayed items.


I understand.

 Are there some rules where this is described?



Not really. I do think readability is relevant; I'm not really a fan of 
the old 72 column limit but I do think there should be some limit.

The above notwithstanding, I can revert the hf[] reformatting if you 
desire.


Bill
___
Sent via:Wireshark-dev mailing list wireshark-dev@wireshark.org
Archives:http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
 mailto:wireshark-dev-requ...@wireshark.org?subject=unsubscribe

___
Sent via:Wireshark-dev mailing list wireshark-dev@wireshark.org
Archives:http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
 mailto:wireshark-dev-requ...@wireshark.org?subject=unsubscribe


[Wireshark-dev] r54005 by wmeier for packet-mq.c and packet-mq-pcf.c

2013-12-13 Thread RobiOneKenobi
Reformat hf[] entries ?

 

Why, now all seems no more aligned for me, and your reformat also left
spaces between text and comma.

 

I do not agree with such reformat, as it seems less readable as the one I've
put in.

 

I like to have some parts aligned to an ease of use for to have the same
length of displayed items.

 

Are there some rules where this is described?

 

 

___
Sent via:Wireshark-dev mailing list wireshark-dev@wireshark.org
Archives:http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
 mailto:wireshark-dev-requ...@wireshark.org?subject=unsubscribe

Re: [Wireshark-dev] r54005 by wmeier for packet-mq.c and packet-mq-pcf.c

2013-12-13 Thread Bill Meier

On 12/13/2013 12:21 PM, RobiOneKenobi wrote:

Reformat hf[] entries ?

Why, now all seems no more aligned for me, and your reformat also
left spaces between text and comma.


You're right; I was sloppy about leaving spaces between the text and the 
comma.   :)




I do not agree with such reformat, as it seems less readable as the
one I’ve put in.



I made the change because, personally, I find the quite long lines quite
difficult to read when they when they exceed the width of my screen
(which I expect they will do on screens used by many).


I like to have some parts aligned to an ease of use for to have the
same length of displayed items.



I understand.


Are there some rules where this is described?




Not really. I do think readability is relevant; I'm not really a fan of 
the old 72 column limit but I do think there should be some limit.


The above notwithstanding, I can revert the hf[] reformatting if you 
desire.



Bill
___
Sent via:Wireshark-dev mailing list wireshark-dev@wireshark.org
Archives:http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
mailto:wireshark-dev-requ...@wireshark.org?subject=unsubscribe