TEST_CODEOWNERS/rails
3
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

Merge pull request #6074 from mark-rushakoff/unused-variables

Browse Source 
Remove some unused variable assignments
This commit is contained in:
Jeremy Kemper 2012-04-29 21:27:09 -07:00
commit 363d7ee551
9 changed files with 7 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
actionpack/lib/action_dispatch/middleware/cookies.rb
View File

@ -339,7 +339,6 @@ def initialize(app)
end
def call(env)
cookie_jar = nil
status, headers, body = @app.call(env)
if cookie_jar = env['action_dispatch.cookies']

2
actionpack/test/template/date_helper_test.rb
View File

@ -587,7 +587,7 @@ def test_select_minute_with_hidden_and_field_name
end
def test_select_minute_with_html_options
expected = expected = %(<select id="date_minute" name="date[minute]" class="selector" accesskey="M">\n)
expected = %(<select id="date_minute" name="date[minute]" class="selector" accesskey="M">\n)
expected << %(<option value="00">00</option>\n<option value="01">01</option>\n<option value="02">02</option>\n<option value="03">03</option>\n<option value="04" selected="selected">04</option>\n<option value="05">05</option>\n<option value="06">06</option>\n<option value="07">07</option>\n<option value="08">08</option>\n<option value="09">09</option>\n<option value="10">10</option>\n<option value="11">11</option>\n<option value="12">12</option>\n<option value="13">13</option>\n<option value="14">14</option>\n<option value="15">15</option>\n<option value="16">16</option>\n<option value="17">17</option>\n<option value="18">18</option>\n<option value="19">19</option>\n<option value="20">20</option>\n<option value="21">21</option>\n<option value="22">22</option>\n<option value="23">23</option>\n<option value="24">24</option>\n<option value="25">25</option>\n<option value="26">26</option>\n<option value="27">27</option>\n<option value="28">28</option>\n<option value="29">29</option>\n<option value="30">30</option>\n<option value="31">31</option>\n<option value="32">32</option>\n<option value="33">33</option>\n<option value="34">34</option>\n<option value="35">35</option>\n<option value="36">36</option>\n<option value="37">37</option>\n<option value="38">38</option>\n<option value="39">39</option>\n<option value="40">40</option>\n<option value="41">41</option>\n<option value="42">42</option>\n<option value="43">43</option>\n<option value="44">44</option>\n<option value="45">45</option>\n<option value="46">46</option>\n<option value="47">47</option>\n<option value="48">48</option>\n<option value="49">49</option>\n<option value="50">50</option>\n<option value="51">51</option>\n<option value="52">52</option>\n<option value="53">53</option>\n<option value="54">54</option>\n<option value="55">55</option>\n<option value="56">56</option>\n<option value="57">57</option>\n<option value="58">58</option>\n<option value="59">59</option>\n)
expected << "</select>\n"

8
activerecord/lib/active_record/connection_adapters/abstract/connection_pool.rb
View File

@ -286,12 +286,10 @@ def reap
private
def release(conn)
thread_id = nil
if @reserved_connections[current_connection_id] == conn
thread_id = current_connection_id
thread_id = if @reserved_connections[current_connection_id] == conn
current_connection_id
else
thread_id = @reserved_connections.keys.find { |k|
@reserved_connections.keys.find { |k|
@reserved_connections[k] == conn
}
end

1
activerecord/test/cases/associations/eager_load_nested_include_test.rb
View File

@ -92,7 +92,6 @@ def generate_test_object_graphs
end
def test_include_query
res = 0
res = ShapeExpression.scoped(:includes => [ :shape, { :paint => :non_poly } ]).all
assert_equal NUM_SHAPE_EXPRESSIONS, res.size
assert_queries(0) do

4
activerecord/test/cases/attribute_methods_test.rb
View File

@ -481,7 +481,7 @@ def test_should_unserialize_attributes_for_frozen_records
end
def test_typecast_attribute_from_select_to_false
topic = Topic.create(:title => 'Budget')
Topic.create(:title => 'Budget')
# Oracle does not support boolean expressions in SELECT
if current_adapter?(:OracleAdapter)
topic = Topic.scoped(:select => "topics.*, 0 as is_test").first
@ -492,7 +492,7 @@ def test_typecast_attribute_from_select_to_false
end
def test_typecast_attribute_from_select_to_true
topic = Topic.create(:title => 'Budget')
Topic.create(:title => 'Budget')
# Oracle does not support boolean expressions in SELECT
if current_adapter?(:OracleAdapter)
topic = Topic.scoped(:select => "topics.*, 1 as is_test").first

2
activerecord/test/cases/invalid_date_test.rb
View File

@ -7,8 +7,6 @@ def test_assign_valid_dates
invalid_dates = [[2007, 11, 31], [1993, 2, 29], [2007, 2, 29]]
topic = Topic.new
valid_dates.each do |date_src|
topic = Topic.new("last_read(1i)" => date_src[0].to_s, "last_read(2i)" => date_src[1].to_s, "last_read(3i)" => date_src[2].to_s)
# Oracle DATE columns are datetime columns and Oracle adapter returns Time value

1
activerecord/test/cases/persistence_test.rb
View File

@ -405,7 +405,6 @@ def test_update_attribute_for_readonly_attribute
def test_update_attribute_with_one_updated
t = Topic.first
title = t.title
t.update_attribute(:title, 'super_title')
assert_equal 'super_title', t.title
assert !t.changed?, "topic should not have changed"

2
activesupport/lib/active_support/testing/performance/jruby.rb
View File

@ -42,7 +42,7 @@ def record
klasses.each do |klass|
fname = output_filename(klass)
FileUtils.mkdir_p(File.dirname(fname))
file = File.open(fname, 'wb') do |file|
File.open(fname, 'wb') do |file|
klass.new(@data).printProfile(file)
end
end

1
railties/lib/rails/commands/runner.rb
View File

@ -9,7 +9,6 @@
end
ARGV.clone.options do |opts|
script_name = File.basename($0)
opts.banner = "Usage: runner [options] ('Some.ruby(code)' or a filename)"
opts.separator ""