Image handler

<a href='#crh-start'></a><a href='#crh-data-%7B%22processed%22%3A%20%5B%22https%3A//github.com/qcode-software/qcode-tcl/pull/107%23discussion_r20791685%22%5D%2C%20%22comments%22%3A%20%7B%22Pull%20b17779064c1437b02a27b24e4655c70a3cd6bde7%20tcl/image_file.tcl%205%22%3A%20%7B%22html_url%22%3A%20%22https%3A//github.com/qcode-software/qcode-tcl/pull/107%23discussion_r20791685%22%2C%20%22comments%22%3A%20%5B%7B%22body%22%3A%20%22could%20we%20use%20%60max_width%60%20rather%20than%20%60allowed_max_width%60%20%22%2C%20%22created_at%22%3A%20%222014-11-24T14%3A17%3A38Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/486212%3Fv%3D3%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/bvw%22%7D%7D%5D%2C%20%22title%22%3A%20%22File%3A%20tcl/image_file.tcl%3AL159-171%22%7D%7D%7D'></a>
<a href='https://www.codereviewhub.com/'><img src='http://www.codereviewhub.com/site/github-bar.png' height=40></a>
- [ ] <a href='#crh-comment-Pull b17779064c1437b02a27b24e4655c70a3cd6bde7 tcl/image_file.tcl 5'></a> <img src='http://www.codereviewhub.com/site/github-remaining.png' height=16 width=60>&nbsp;<b><a href='https://github.com/qcode-software/qcode-tcl/pull/107#discussion_r20791685'>File: tcl/image_file.tcl:L159-171</a></b>
- <a href='https://github.com/bvw'><img border=0 src='https://avatars.githubusercontent.com/u/486212?v=3' height=16 width=16'></a> could we use `max_width` rather than `allowed_max_width`


<a href='https://www.codereviewhub.com/qcode-software/qcode-tcl/pull/107?mark_as_completed=1'><img src='http://www.codereviewhub.com/site/github-mark-as-completed.png' height=26></a>&nbsp;<a href='https://www.codereviewhub.com/qcode-software/qcode-tcl/pull/107?approve=1'><img src='http://www.codereviewhub.com/site/github-approve.png' height=26></a>&nbsp;<a href='https://github.com/qcode-software/qcode-tcl/pull/107'><img src='http://www.codereviewhub.com/site/github-refresh.png' height=26></a>
<a href='#crh-end'></a>
just using max_width would collide with max_width of the requested image
size

On 24 November 2014 at 14:17, Bernhard van Woerden <notifications@github.com
> wrote:

> In tcl/image_file.tcl:
>
> > @@ -159,11 +159,12 @@ proc qc::image_data {cache_dir file_id max_width max_height} {
> >      return [qc::image_cache_data $cache_dir $file_id $max_width $max_height]
> >  }
> >
> > -proc qc::image_handler {cache_dir {image_redirect_handler UNDEF}} {
> > +proc qc::image_handler {cache_dir {image_redirect_handler "qc::image_redirect_handler"} {allowed_max_width 2560} {allowed_max_height 2560}} {
>
> could we use max_width rather than allowed_max_width
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/qcode-software/qcode-tcl/pull/107/files#r20791685>.
>



-- 
Kind Regards,
Daniel Clark
Qcode Software Limited
www.qcode.co.uk
Tel: +44(0) 1463 896 482