Plugin: MD Detect Page Type. Pagination, categories, yearly archives and single entries all on one template.

I’m currently using this plugin as I really want to keep my urls clean. So far it works great!

My only concern is the overhead that might come from the conditionals. In your example code it appears the conditionals are setup as advanced conditionals?

{exp:md_detect_page_type url_segment="{segment_2}"}
    {!-- if segment_2 is empty or it equals P## --}
    {if pagination_page || segment_2 == ""}

I’m wondering if this should be changed to the following to create a simple conditional on the inside to prevent futher parsing if segment_2 is not empty

{exp:md_detect_page_type url_segment="{segment_2}"}
    {!-- if segment_2 is empty or it equals P## --}
    {if pagination_page== ""}
       {if segment_2 == ""}    {!-- This sets up the simple conditional --}

Whats your thoughts on this… Would it improve page parsing?