openamf/src/java/org/openamf/io AMFDeserializer.java,1.19,1.20

[email protected]
Newsgroups gmane.comp.java.openamf.cvs
Message-ID <[email protected]>
Update of /cvsroot/openamf/openamf/src/java/org/openamf/io
In directory sc8-pr-cvs1:/tmp/cvs-serv16704/src/java/org/openamf/io

Modified Files:
	AMFDeserializer.java 
Log Message:
fixed bug with null values

Index: AMFDeserializer.java
===================================================================
RCS file: /cvsroot/openamf/openamf/src/java/org/openamf/io/AMFDeserializer.java,v
retrieving revision 1.19
retrieving revision 1.20
diff -C2 -d -r1.19 -r1.20
*** AMFDeserializer.java	6 Jul 2003 17:25:51 -0000	1.19
--- AMFDeserializer.java	17 Jul 2003 22:48:20 -0000	1.20
***************
*** 147,159 ****
  			Object value = readData(type);
  			// Save the name/value pair in the map
! 			aso.put(key, value);
! 			log.debug(
! 				" adding {key="
! 					+ key
! 					+ ", value="
! 					+ value
! 					+ ", type="
! 					+ type
! 					+ "}");
  			// Get the next name
  			key = inputStream.readUTF();
--- 147,163 ----
  			Object value = readData(type);
  			// Save the name/value pair in the map
! 			if (value == null) {
! 				log.warn("Skipping NULL value for :" + key);
! 			} else {
! 				aso.put(key, value);
! 				log.debug(
! 					" adding {key="
! 						+ key
! 						+ ", value="
! 						+ value
! 						+ ", type="
! 						+ type
! 						+ "}");
! 			}
  			// Get the next name
  			key = inputStream.readUTF();




-------------------------------------------------------
This SF.net email is sponsored by: VM Ware
With VMware you can run multiple operating systems on a single machine.
WITHOUT REBOOTING! Mix Linux / Windows / Novell virtual machines at the
same time. Free trial click here: http://www.vmware.com/wl/offer/345/0
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.