Skip to content

Commit

Permalink
Upgrade JRuby parser code to fix deprecation
Browse files Browse the repository at this point in the history
  • Loading branch information
nicksieger committed Jan 22, 2010
1 parent 1d17406 commit d9479cb
Show file tree
Hide file tree
Showing 4 changed files with 49 additions and 72 deletions.
4 changes: 2 additions & 2 deletions ext/http11/http11_parser.java.rl
Original file line number Diff line number Diff line change
Expand Up @@ -116,12 +116,12 @@ public class Http11Parser {
public int execute(ByteList buffer, int off) {
int p, pe;
int cs = parser.cs;
int len = buffer.realSize;
int len = buffer.length();
assert off<=len : "offset past end of buffer";

p = off;
pe = len;
byte[] data = buffer.bytes;
byte[] data = buffer.unsafeBytes();
parser.buffer = buffer;

%% write exec;
Expand Down
97 changes: 36 additions & 61 deletions ext/http11/org/jruby/mongrel/Http11.java
Original file line number Diff line number Diff line change
@@ -1,30 +1,3 @@
/***** BEGIN LICENSE BLOCK *****
* Version: CPL 1.0/GPL 2.0/LGPL 2.1
*
* The contents of this file are subject to the Common Public
* License Version 1.0 (the "License"); you may not use this file
* except in compliance with the License. You may obtain a copy of
* the License at http://www.eclipse.org/legal/cpl-v10.html
*
* Software distributed under the License is distributed on an "AS
* IS" basis, WITHOUT WARRANTY OF ANY KIND, either express or
* implied. See the License for the specific language governing
* rights and limitations under the License.
*
* Copyright (C) 2007 Ola Bini <[email protected]>
*
* Alternatively, the contents of this file may be used under the terms of
* either of the GNU General Public License Version 2 or later (the "GPL"),
* or the GNU Lesser General Public License Version 2.1 or later (the "LGPL"),
* in which case the provisions of the GPL or the LGPL are applicable instead
* of those above. If you wish to allow use of your version of this file only
* under the terms of either the GPL or the LGPL, and not to allow others to
* use your version of this file under the terms of the CPL, indicate your
* decision by deleting the provisions above and replace them with the notice
* and other provisions required by the GPL or the LGPL. If you do not delete
* the provisions above, a recipient may use your version of this file under
* the terms of any one of the CPL, the GPL or the LGPL.
***** END LICENSE BLOCK *****/
package org.jruby.mongrel;

import org.jruby.Ruby;
Expand All @@ -35,8 +8,10 @@
import org.jruby.RubyObject;
import org.jruby.RubyString;

import org.jruby.runtime.CallbackFactory;
import org.jruby.anno.JRubyMethod;

import org.jruby.runtime.ObjectAllocator;
import org.jruby.runtime.ThreadContext;
import org.jruby.runtime.builtin.IRubyObject;

import org.jruby.exceptions.RaiseException;
Expand Down Expand Up @@ -73,16 +48,8 @@ public static void createHttp11(Ruby runtime) {
RubyModule mMongrel = runtime.defineModule("Mongrel");
mMongrel.defineClassUnder("HttpParserError",runtime.getClass("IOError"),runtime.getClass("IOError").getAllocator());

CallbackFactory cf = runtime.callbackFactory(Http11.class);

RubyClass cHttpParser = mMongrel.defineClassUnder("HttpParser",runtime.getObject(),ALLOCATOR);
cHttpParser.defineFastMethod("initialize",cf.getFastMethod("initialize"));
cHttpParser.defineFastMethod("reset",cf.getFastMethod("reset"));
cHttpParser.defineFastMethod("finish",cf.getFastMethod("finish"));
cHttpParser.defineFastMethod("execute",cf.getFastMethod("execute", IRubyObject.class, IRubyObject.class, IRubyObject.class));
cHttpParser.defineFastMethod("error?",cf.getFastMethod("has_error"));
cHttpParser.defineFastMethod("finished?",cf.getFastMethod("is_finished"));
cHttpParser.defineFastMethod("nread",cf.getFastMethod("nread"));
cHttpParser.defineAnnotatedMethods(Http11.class);
}

private Ruby runtime;
Expand Down Expand Up @@ -121,23 +88,23 @@ public void call(Object data, int field, int flen, int value, int vlen) {
v = RubyString.newString(runtime, new ByteList(Http11.this.hp.parser.buffer,value,vlen));
f = RubyString.newString(runtime, "HTTP_");
ByteList b = new ByteList(Http11.this.hp.parser.buffer,field,flen);
for(int i=0,j=b.realSize;i<j;i++) {
if((b.bytes[i]&0xFF) == '-') {
b.bytes[i] = (byte)'_';
for(int i = 0,j = b.length();i<j;i++) {
if((b.get(i) & 0xFF) == '-') {
b.set(i, (byte)'_');
} else {
b.bytes[i] = (byte)Character.toUpperCase((char)b.bytes[i]);
b.set(i, (byte)Character.toUpperCase((char)b.get(i)));
}
}
f.cat(b);
req.aset(f,v);
req.op_aset(req.getRuntime().getCurrentContext(), f,v);
}
};

private Http11Parser.ElementCB request_method = new Http11Parser.ElementCB() {
public void call(Object data, int at, int length) {
RubyHash req = (RubyHash)data;
RubyString val = RubyString.newString(runtime,new ByteList(hp.parser.buffer,at,length));
req.aset(runtime.newString("REQUEST_METHOD"),val);
req.op_aset(req.getRuntime().getCurrentContext(), runtime.newString("REQUEST_METHOD"),val);
}
};

Expand All @@ -146,7 +113,7 @@ public void call(Object data, int at, int length) {
RubyHash req = (RubyHash)data;
validateMaxLength(length, MAX_REQUEST_URI_LENGTH, MAX_REQUEST_URI_LENGTH_ERR);
RubyString val = RubyString.newString(runtime,new ByteList(hp.parser.buffer,at,length));
req.aset(runtime.newString("REQUEST_URI"),val);
req.op_aset(req.getRuntime().getCurrentContext(), runtime.newString("REQUEST_URI"),val);
}
};

Expand All @@ -155,7 +122,7 @@ public void call(Object data, int at, int length) {
RubyHash req = (RubyHash)data;
validateMaxLength(length, MAX_FRAGMENT_LENGTH, MAX_FRAGMENT_LENGTH_ERR);
RubyString val = RubyString.newString(runtime,new ByteList(hp.parser.buffer,at,length));
req.aset(runtime.newString("FRAGMENT"),val);
req.op_aset(req.getRuntime().getCurrentContext(), runtime.newString("FRAGMENT"),val);
}
};

Expand All @@ -164,7 +131,7 @@ public void call(Object data, int at, int length) {
RubyHash req = (RubyHash)data;
validateMaxLength(length, MAX_REQUEST_PATH_LENGTH, MAX_REQUEST_PATH_LENGTH_ERR);
RubyString val = RubyString.newString(runtime,new ByteList(hp.parser.buffer,at,length));
req.aset(runtime.newString("REQUEST_PATH"),val);
req.op_aset(req.getRuntime().getCurrentContext(), runtime.newString("REQUEST_PATH"),val);
}
};

Expand All @@ -173,72 +140,77 @@ public void call(Object data, int at, int length) {
RubyHash req = (RubyHash)data;
validateMaxLength(length, MAX_QUERY_STRING_LENGTH, MAX_QUERY_STRING_LENGTH_ERR);
RubyString val = RubyString.newString(runtime,new ByteList(hp.parser.buffer,at,length));
req.aset(runtime.newString("QUERY_STRING"),val);
req.op_aset(req.getRuntime().getCurrentContext(), runtime.newString("QUERY_STRING"),val);
}
};

private Http11Parser.ElementCB http_version = new Http11Parser.ElementCB() {
public void call(Object data, int at, int length) {
RubyHash req = (RubyHash)data;
RubyString val = RubyString.newString(runtime,new ByteList(hp.parser.buffer,at,length));
req.aset(runtime.newString("HTTP_VERSION"),val);
req.op_aset(req.getRuntime().getCurrentContext(), runtime.newString("HTTP_VERSION"),val);
}
};

private Http11Parser.ElementCB header_done = new Http11Parser.ElementCB() {
public void call(Object data, int at, int length) {
RubyHash req = (RubyHash)data;
ThreadContext context = req.getRuntime().getCurrentContext();
IRubyObject temp,ctype,clen;

clen = req.aref(runtime.newString("HTTP_CONTENT_LENGTH"));
clen = req.op_aref(context, runtime.newString("HTTP_CONTENT_LENGTH"));
if(!clen.isNil()) {
req.aset(runtime.newString("CONTENT_LENGTH"),clen);
req.op_aset(context, runtime.newString("CONTENT_LENGTH"),clen);
}

ctype = req.aref(runtime.newString("HTTP_CONTENT_TYPE"));
ctype = req.op_aref(context, runtime.newString("HTTP_CONTENT_TYPE"));
if(!ctype.isNil()) {
req.aset(runtime.newString("CONTENT_TYPE"),ctype);
req.op_aset(context, runtime.newString("CONTENT_TYPE"),ctype);
}

req.aset(runtime.newString("GATEWAY_INTERFACE"),runtime.newString("CGI/1.2"));
if(!(temp = req.aref(runtime.newString("HTTP_HOST"))).isNil()) {
req.op_aset(context, runtime.newString("GATEWAY_INTERFACE"),runtime.newString("CGI/1.2"));
if(!(temp = req.op_aref(context, runtime.newString("HTTP_HOST"))).isNil()) {
String s = temp.toString();
int colon = s.indexOf(':');
if(colon != -1) {
req.aset(runtime.newString("SERVER_NAME"),runtime.newString(s.substring(0,colon)));
req.aset(runtime.newString("SERVER_PORT"),runtime.newString(s.substring(colon+1)));
req.op_aset(context, runtime.newString("SERVER_NAME"),runtime.newString(s.substring(0,colon)));
req.op_aset(context, runtime.newString("SERVER_PORT"),runtime.newString(s.substring(colon+1)));
} else {
req.aset(runtime.newString("SERVER_NAME"),temp);
req.aset(runtime.newString("SERVER_PORT"),runtime.newString("80"));
req.op_aset(context, runtime.newString("SERVER_NAME"),temp);
req.op_aset(context, runtime.newString("SERVER_PORT"),runtime.newString("80"));
}
}

req.setInstanceVariable("@http_body", RubyString.newString(runtime, new ByteList(hp.parser.buffer, at, length)));
req.aset(runtime.newString("SERVER_PROTOCOL"),runtime.newString("HTTP/1.1"));
req.aset(runtime.newString("SERVER_SOFTWARE"),runtime.newString("Mongrel 1.1.6"));
req.op_aset(context, runtime.newString("SERVER_PROTOCOL"),runtime.newString("HTTP/1.1"));
req.op_aset(context, runtime.newString("SERVER_SOFTWARE"),runtime.newString("Mongrel 1.1.6"));
}
};

@JRubyMethod
public IRubyObject initialize() {
this.hp.parser.init();
return this;
}

@JRubyMethod
public IRubyObject reset() {
this.hp.parser.init();
return runtime.getNil();
}

@JRubyMethod
public IRubyObject finish() {
this.hp.finish();
return this.hp.is_finished() ? runtime.getTrue() : runtime.getFalse();
}

@JRubyMethod
public IRubyObject execute(IRubyObject req_hash, IRubyObject data, IRubyObject start) {
int from = 0;
from = RubyNumeric.fix2int(start);
ByteList d = ((RubyString)data).getByteList();
if(from >= d.realSize) {
if(from >= d.length()) {
throw new RaiseException(runtime, eHttpParserError, "Requested start is after data buffer end.", true);
} else {
this.hp.parser.data = req_hash;
Expand All @@ -252,14 +224,17 @@ public IRubyObject execute(IRubyObject req_hash, IRubyObject data, IRubyObject s
}
}

@JRubyMethod(name = "error?")
public IRubyObject has_error() {
return this.hp.has_error() ? runtime.getTrue() : runtime.getFalse();
}

@JRubyMethod(name = "finished?")
public IRubyObject is_finished() {
return this.hp.is_finished() ? runtime.getTrue() : runtime.getFalse();
}

@JRubyMethod
public IRubyObject nread() {
return runtime.newFixnum(this.hp.parser.nread);
}
Expand Down
12 changes: 6 additions & 6 deletions ext/http11/org/jruby/mongrel/Http11Parser.java
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ public class Http11Parser {

/** Data **/

// line 18 "ext/http11_java/org/jruby/mongrel/Http11Parser.java"
// line 18 "ext/http11/org/jruby/mongrel/Http11Parser.java"
private static byte[] init__http_parser_actions_0()
{
return new byte [] {
Expand Down Expand Up @@ -236,7 +236,7 @@ public void init() {
cs = 0;


// line 240 "ext/http11_java/org/jruby/mongrel/Http11Parser.java"
// line 240 "ext/http11/org/jruby/mongrel/Http11Parser.java"
{
cs = http_parser_start;
}
Expand All @@ -257,16 +257,16 @@ public void init() {
public int execute(ByteList buffer, int off) {
int p, pe;
int cs = parser.cs;
int len = buffer.realSize;
int len = buffer.length();
assert off<=len : "offset past end of buffer";

p = off;
pe = len;
byte[] data = buffer.bytes;
byte[] data = buffer.unsafeBytes();
parser.buffer = buffer;


// line 270 "ext/http11_java/org/jruby/mongrel/Http11Parser.java"
// line 270 "ext/http11/org/jruby/mongrel/Http11Parser.java"
{
int _klen;
int _trans = 0;
Expand Down Expand Up @@ -431,7 +431,7 @@ else if ( data[p] > _http_parser_trans_keys[_mid+1] )
{ p += 1; _goto_targ = 5; if (true) continue _goto;}
}
break;
// line 435 "ext/http11_java/org/jruby/mongrel/Http11Parser.java"
// line 435 "ext/http11/org/jruby/mongrel/Http11Parser.java"
}
}
}
Expand Down
8 changes: 5 additions & 3 deletions tasks/ragel.rake
Original file line number Diff line number Diff line change
Expand Up @@ -9,10 +9,12 @@ file 'ext/http11/http11_parser.c' => ['ext/http11/http11_parser.rl'] do |t|
end
end

file 'ext/http11_java/org/jruby/mongrel/Http11Parser.java' => ['ext/http11/http11_parser.rl'] do |t|
file 'ext/http11/org/jruby/mongrel/Http11Parser.java' => ['ext/http11/http11_parser.java.rl'] do |t|
begin
sh "ragel #{t.prerequisites.last} -J -o #{t.name}"
sh "ragel #{t.prerequisites.last} -J -G2 -o #{t.name}"
rescue
fail "Could not build wrapper using Ragel (it failed or not installed?)"
fail "Could not build wrapper using Ragel (it failed or not installed?)"
end
end

task :ragel => (defined?(JRUBY_VERSION) ? 'ext/http11/org/jruby/mongrel/Http11Parser.java' : 'ext/http11/http11_parser.c')

0 comments on commit d9479cb

Please sign in to comment.