[sdcc-devel] [ sdcc-Patches-3574556 ] Compressed pic16devices.txt file

[SourceForge.net <no...@so...>]

Patches item #3574556, was opened at 2012-10-04 12:59
Message generated for change (Comment added) made by borutr
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=300599&aid=3574556&group_id=599

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
>Status: Pending
>Resolution: Fixed
Priority: 5
Private: No
Submitted By: Molnár Károly (molnarkaroly)
>Assigned to: Borut Ražem (borutr)
Summary: Compressed pic16devices.txt file

Initial Comment:
This pic16devices.txt version is free from the repetitions and therefore smaller. I give him a utility. This performs the compression and decompression.
Thus it is advisable to use:
  1.) Uncompress the file.
  2.) Modify the file.
  3.) Compress the file.


----------------------------------------------------------------------

>Comment By: Borut Ražem (borutr)
Date: 2012-10-05 06:06

Message:
Slightly modified patch applied in svn revision #8134.

Karoly, I think that it would be more appropriate to use word "optimized"
instead of "compressed" or even "zipped" which IMHO is misleading. So name
of the utility wold be optimize-pic16devices.pl. If you agree I can rename
the utility and make all the changes in the file content.

Borut

----------------------------------------------------------------------

Comment By: Molnár Károly (molnarkaroly)
Date: 2012-10-05 05:01

Message:
In the pic16devices.txt file the header at the end of this line is
unnecessary: "# The compression was performed the zip-pic16devices.pl
program." I deleted from the utility the two lines which create this text.
After all decompression or compression, two new lines would have increased
the length of header. This would have been unnecessary and confusing.

----------------------------------------------------------------------

Comment By: Molnár Károly (molnarkaroly)
Date: 2012-10-05 03:03

Message:
It would make sense, to use the new one everywhere. Mainly because the
processors are sorted by name. It is easier to navigate through in the
file. After the "uncompression", is much easier to perform any
modifications. After that can be again compress. (Of course only if they
there is beside him of the utility as well.)

----------------------------------------------------------------------

Comment By: Borut Ražem (borutr)
Date: 2012-10-04 14:04

Message:
Karoly,

what is your opinion: should we keep the old pic16devices.txt file and
include the "compressed" one  only in the binary packages or can we
completely replace it wit the new "compressed" one?

I don't see much (or any) difference between them regarding the
readability, so IMHO new one can replace the old one.

Borut

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=300599&aid=3574556&group_id=599