Various small fixes for julia-0.6

* Remove depwarns for takebuf_string, typealias, Base.linearindexing
* Use Compat.readline for new chomp argument
* Fix tests which were broken by simplified array arithmetic promotion

Author: c42f

REQUIRE

@@ -1 +1,2 @@
 julia 0.5
+Compat 0.13.0

src/PlyIO.jl

@@ -2,6 +2,8 @@ __precompile__()
 
 module PlyIO
 
+using Compat
+
 # Types for the ply data model
 export Ply, PlyElement, PlyComment, ArrayProperty, ListProperty
 export plyname  # Is there something in base we could overload for this?

src/io.jl

@@ -44,7 +44,7 @@ ply_type_name(::Int64)    = "int64"
 ply_type_name(::Float32)  = "float32"
 ply_type_name(::Float64)  = "float64"
 
-typealias PlyNativeType Union{UInt8,UInt16,UInt32,UInt64,Int8,Int16,Int32,Int64,Float32,Float64}
+@compat const PlyNativeType = Union{UInt8,UInt16,UInt32,UInt64,Int8,Int16,Int32,Int64,Float32,Float64}
 
 ply_type_name{T<:PlyNativeType}(A::AbstractArray{T}) = ply_type_name(T)
 
@@ -56,23 +56,28 @@ const _host_is_little_endian = (ENDIAN_BOM == 0x04030201)
 
 
 function read_header(ply_file)
-    @assert readline(ply_file) == "ply\n"
+    firstline = Compat.readline(ply_file)
+    if firstline != "ply"
+        throw(ErrorException("Expected \"ply\" header, got \"$firstline\""))
+    end
     element_name = ""
     element_numel = 0
     element_props = Vector{AbstractVector}()
     elements = PlyElement[]
     comments = PlyComment[]
     format = nothing
     while true
-        line = strip(readline(ply_file))
+        line = Compat.readline(ply_file)
         if line == "end_header"
             break
         elseif startswith(line, "comment")
             push!(comments, PlyComment(strip(line[8:end]), length(elements)+1))
         elseif startswith(line, "format")
             tok, format_type, format_version = split(line)
             @assert tok == "format"
-            @assert format_version == "1.0"
+            if format_version != "1.0"
+                throw(ErrorException("Expected ply version 1.0, got $format_version"))
+            end
             format = format_type == "ascii"                ? Format_ascii :
                      format_type == "binary_little_endian" ? Format_binary_little :
                      format_type == "binary_big_endian"    ? Format_binary_big :

src/types.jl

@@ -9,7 +9,7 @@ Return the name that `data` is associated with when serialized in a ply file
 function plyname
 end
 
-typealias PropNameList Union{AbstractVector,Tuple}
+@compat const PropNameList = Union{AbstractVector,Tuple}
 
 #--------------------------------------------------
 """
@@ -41,7 +41,7 @@ Base.summary(prop::ArrayProperty) = "$(length(prop))-element $(typeof(prop)) \"$
 Base.size(prop::ArrayProperty) = size(prop.data)
 Base.getindex(prop::ArrayProperty, i::Int) = prop.data[i]
 Base.setindex!(prop::ArrayProperty, v, i::Int) = prop.data[i] = v
-Base.linearindexing(prop::ArrayProperty) = Base.LinearFast()
+@compat Base.IndexStyle(::Type{<:ArrayProperty}) = IndexLinear()
 
 # List methods
 Base.resize!(prop::ArrayProperty, len) = resize!(prop.data, len)
@@ -80,7 +80,7 @@ Base.summary(prop::ListProperty) = "$(length(prop))-element $(typeof(prop)) \"$(
 Base.length(prop::ListProperty) = length(prop.start_inds)-1
 Base.size(prop::ListProperty) = (length(prop),)
 Base.getindex(prop::ListProperty, i::Int) = prop.data[prop.start_inds[i]:prop.start_inds[i+1]-1]
-Base.linearindexing(prop::ListProperty) = Base.LinearFast()
+@compat Base.IndexStyle(::Type{<:ListProperty}) = IndexLinear()
 # TODO: Do we need Base.setindex!() ?  Hard to provide with above formulation...
 
 # List methods
@@ -198,7 +198,7 @@ Ply() = Ply(Vector{PlyElement}(), Vector{String}())
 function Base.show(io::IO, ply::Ply)
     buf = IOBuffer()
     write_header(ply, buf, true)
-    headerstr = takebuf_string(buf)
+    headerstr = String(take!(buf))
     headerstr = replace(strip(headerstr), "\n", "\n ")
     print(io, "$Ply with header:\n $headerstr")
 end

test/runtests.jl

@@ -1,6 +1,7 @@
 using PlyIO
 using StaticArrays
 using Base.Test
+using Compat
 
 @testset "PlyIO" begin
 
@@ -22,7 +23,7 @@ using Base.Test
 
     buf = IOBuffer()
     save_ply(ply, buf, ascii=true)
-    str = takebuf_string(buf)
+    str = String(take!(buf))
     open("simple_test_tmp.ply", "w") do fid
         write(fid, str)
     end
@@ -58,7 +59,7 @@ end
 
     buf = IOBuffer()
     save_ply(ply, buf, ascii=true)
-    str = takebuf_string(buf)
+    str = String(take!(buf))
     open("empty_test_tmp.ply", "w") do fid
         write(fid, str)
     end
@@ -114,7 +115,7 @@ end
                                                    UInt8, UInt16, UInt32, UInt64,
                                                    Float32, Float64]
         ply = Ply()
-        arrayprop = zeros(proptype, 1) + 42
+        arrayprop = fill(proptype(42), 1)
         listprop = ListProperty("listprop", proptype<:Integer ? proptype : Int32, proptype)
         push!(listprop, collect(proptype, 1:10))
         push!(ply, PlyElement("test", ArrayProperty("arrayprop", arrayprop), listprop))
@@ -141,7 +142,7 @@ end
                          ))
     buf = IOBuffer()
     save_ply(ply, buf, ascii=true)
-    str = takebuf_string(buf)
+    str = String(take!(buf))
     open("SVector_properties_test_tmp.ply", "w") do fid
         write(fid, str)
     end