rulinux.net - Форум - Rulinux.net - Прием патчей к исходникам LOR-NG http://rulinux.net/ rulinux.net - Форум - Rulinux.net - Прием патчей к исходникам LOR-NG http://rulinux.net/ http://rulinux.net/rss_icon.png Re:Прием патчей к исходникам LOR-NG https://rulinux.net/message.php?newsid=1588&page=1#12580 https://rulinux.net/message.php?newsid=1588&page=1#12580 Fri, 22 May 2009 03:14:46 +0400 > Повыбрасывать из кода в classes/*.class.php вызовы "mysql_close();" - насколько я понимаю, из-за этих вызовов каждый вызов метода класса будет переустанавливать соединение к БД.

не, так было давно. Сейчас в действительности имеется пул соединений, и при запросе соединения с бд новое обычно не создаётся, а используется из пула. При mysql_close() соединение не закрывается, а возвращается в пул и может быть повторно использовано при новом запросе. Убирание вызовов mysql_close() в действительности мало на что повлияет, так как по завершении скрипта все его открытытые дескрипторы всё равно закрываются.

]]> Re:Прием патчей к исходникам LOR-NG https://rulinux.net/message.php?newsid=1588&page=1#12579 https://rulinux.net/message.php?newsid=1588&page=1#12579 Tue, 12 May 2009 20:39:25 +0400
  2. #ifndef nsHTMLReflowMetrics_h___
  3. #define nsHTMLReflowMetrics_h___
  5. #include <stdio.h>
  6. #include "nsISupports.h"
  7. #include "nsMargin.h"
  8. #include "nsRect.h"
  9. // for MOZ_MATHML
  10. #include "nsIRenderingContext.h" //to get struct nsBoundingMetrics
  12. //———————————————————————-
  14. // Option flags
  15. #define NS_REFLOW_CALC_MAX_WIDTH 0x0001
  16. #ifdef MOZ_MATHML
  17. #define NS_REFLOW_CALC_BOUNDING_METRICS 0x0002
  18. #endif
  20. /**
  21. * An nsCollapsingMargin represents a vertical collapsing margin between
  22. * blocks as described in section 8.3.1 of CSS2,
  23. * <URL: http://www.w3.org/TR/REC-CSS2/box.html#collapsing-margins >.
  24. *
  25. * All adjacent vertical margins collapse, and the resulting margin is
  26. * the sum of the largest positive margin included and the smallest (most
  27. * negative) negative margin included.
  28. */
  29. struct nsCollapsingMargin {
  30. private:
  31. nscoord mMostPos; // the largest positive margin included
  32. nscoord mMostNeg; // the smallest negative margin included
  34. public:
  35. nsCollapsingMargin()
  36. : mMostPos(0),
  37. mMostNeg(0)
  38. {
  39. }
  41. nsCollapsingMargin(const nsCollapsingMargin& aOther)
  42. : mMostPos(aOther.mMostPos),
  43. mMostNeg(aOther.mMostNeg)
  44. {
  45. }
  47. nsCollapsingMargin& operator=(const nsCollapsingMargin& aOther)
  48. {
  49. mMostPos = aOther.mMostPos;
  50. mMostNeg = aOther.mMostNeg;
  51. return *this;
  52. }
  54. void Include(nscoord aCoord)
  55. {
  56. if (aCoord > mMostPos)
  57. mMostPos = aCoord;
  58. else if (aCoord < mMostNeg)
  59. mMostNeg = aCoord;
  60. }
  62. void Include(const nsCollapsingMargin& aOther)
  63. {
  64. if (aOther.mMostPos > mMostPos)
  65. mMostPos = aOther.mMostPos;
  66. if (aOther.mMostNeg < mMostNeg)
  67. mMostNeg = aOther.mMostNeg;
  68. }
  70. void Zero()
  71. {
  72. mMostPos = 0;
  73. mMostNeg = 0;
  74. }
  76. PRBool IsZero() const
  77. {
  78. return (mMostPos == 0) && (mMostNeg == 0);
  79. }
  81. nscoord get() const
  82. {
  83. return mMostPos + mMostNeg;
  84. }
  85. };

]]> Re:Прием патчей к исходникам LOR-NG https://rulinux.net/message.php?newsid=1588&page=1#12578 https://rulinux.net/message.php?newsid=1588&page=1#12578 Tue, 12 May 2009 20:38:13 +0400
  2. #ifndef nsHTMLReflowMetrics_h___
  3. #define nsHTMLReflowMetrics_h___
  5. #include <stdio.h>
  6. #include "nsISupports.h"
  7. #include "nsMargin.h"
  8. #include "nsRect.h"
  9. // for MOZ_MATHML
  10. #include "nsIRenderingContext.h" //to get struct nsBoundingMetrics
  12. //———————————————————————-
  14. // Option flags
  15. #define NS_REFLOW_CALC_MAX_WIDTH 0x0001
  16. #ifdef MOZ_MATHML
  17. #define NS_REFLOW_CALC_BOUNDING_METRICS 0x0002
  18. #endif
  20. /**
  21. * An nsCollapsingMargin represents a vertical collapsing margin between
  22. * blocks as described in section 8.3.1 of CSS2,
  23. * <URL: http://www.w3.org/TR/REC-CSS2/box.html#collapsing-margins >.
  24. *
  25. * All adjacent vertical margins collapse, and the resulting margin is
  26. * the sum of the largest positive margin included and the smallest (most
  27. * negative) negative margin included.
  28. */
  29. struct nsCollapsingMargin {
  30. private:
  31. nscoord mMostPos; // the largest positive margin included
  32. nscoord mMostNeg; // the smallest negative margin included
  34. public:
  35. nsCollapsingMargin()
  36. : mMostPos(0),
  37. mMostNeg(0)
  38. {
  39. }
  41. nsCollapsingMargin(const nsCollapsingMargin& aOther)
  42. : mMostPos(aOther.mMostPos),
  43. mMostNeg(aOther.mMostNeg)
  44. {
  45. }
  47. nsCollapsingMargin& operator=(const nsCollapsingMargin& aOther)
  48. {
  49. mMostPos = aOther.mMostPos;
  50. mMostNeg = aOther.mMostNeg;
  51. return *this;
  52. }
  54. void Include(nscoord aCoord)
  55. {
  56. if (aCoord > mMostPos)
  57. mMostPos = aCoord;
  58. else if (aCoord < mMostNeg)
  59. mMostNeg = aCoord;
  60. }
  62. void Include(const nsCollapsingMargin& aOther)
  63. {
  64. if (aOther.mMostPos > mMostPos)
  65. mMostPos = aOther.mMostPos;
  66. if (aOther.mMostNeg < mMostNeg)
  67. mMostNeg = aOther.mMostNeg;
  68. }
  70. void Zero()
  71. {
  72. mMostPos = 0;
  73. mMostNeg = 0;
  74. }
  76. PRBool IsZero() const
  77. {
  78. return (mMostPos == 0) && (mMostNeg == 0);
  79. }
  81. nscoord get() const
  82. {
  83. return mMostPos + mMostNeg;
  84. }
  85. };

]]> Re:Прием патчей к исходникам LOR-NG https://rulinux.net/message.php?newsid=1588&page=1#12577 https://rulinux.net/message.php?newsid=1588&page=1#12577 Tue, 12 May 2009 20:36:56 +0400
  2. #ifndef nsHTMLReflowMetrics_h___
  3. #define nsHTMLReflowMetrics_h___
  5. #include <stdio.h>
  6. #include "nsISupports.h"
  7. #include "nsMargin.h"
  8. #include "nsRect.h"
  9. // for MOZ_MATHML
  10. #include "nsIRenderingContext.h" //to get struct nsBoundingMetrics
  12. //———————————————————————-
  14. // Option flags
  15. #define NS_REFLOW_CALC_MAX_WIDTH 0x0001
  16. #ifdef MOZ_MATHML
  17. #define NS_REFLOW_CALC_BOUNDING_METRICS 0x0002
  18. #endif
  20. /**
  21. * An nsCollapsingMargin represents a vertical collapsing margin between
  22. * blocks as described in section 8.3.1 of CSS2,
  23. * <URL: http://www.w3.org/TR/REC-CSS2/box.html#collapsing-margins >.
  24. *
  25. * All adjacent vertical margins collapse, and the resulting margin is
  26. * the sum of the largest positive margin included and the smallest (most
  27. * negative) negative margin included.
  28. */
  29. struct nsCollapsingMargin {
  30. private:
  31. nscoord mMostPos; // the largest positive margin included
  32. nscoord mMostNeg; // the smallest negative margin included
  34. public:
  35. nsCollapsingMargin()
  36. : mMostPos(0),
  37. mMostNeg(0)
  38. {
  39. }
  41. nsCollapsingMargin(const nsCollapsingMargin& aOther)
  42. : mMostPos(aOther.mMostPos),
  43. mMostNeg(aOther.mMostNeg)
  44. {
  45. }
  47. nsCollapsingMargin& operator=(const nsCollapsingMargin& aOther)
  48. {
  49. mMostPos = aOther.mMostPos;
  50. mMostNeg = aOther.mMostNeg;
  51. return *this;
  52. }
  54. void Include(nscoord aCoord)
  55. {
  56. if (aCoord > mMostPos)
  57. mMostPos = aCoord;
  58. else if (aCoord < mMostNeg)
  59. mMostNeg = aCoord;
  60. }
  62. void Include(const nsCollapsingMargin& aOther)
  63. {
  64. if (aOther.mMostPos > mMostPos)
  65. mMostPos = aOther.mMostPos;
  66. if (aOther.mMostNeg < mMostNeg)
  67. mMostNeg = aOther.mMostNeg;
  68. }
  70. void Zero()
  71. {
  72. mMostPos = 0;
  73. mMostNeg = 0;
  74. }
  76. PRBool IsZero() const
  77. {
  78. return (mMostPos == 0) && (mMostNeg == 0);
  79. }
  81. nscoord get() const
  82. {
  83. return mMostPos + mMostNeg;
  84. }
  85. };

]]> Re:Прием патчей к исходникам LOR-NG https://rulinux.net/message.php?newsid=1588&page=1#12573 https://rulinux.net/message.php?newsid=1588&page=1#12573 Tue, 12 May 2009 17:52:47 +0400 > Да погоди, свн скоро будет уже

Через неделю я уеду в отпуск, а за это время мог бы попробовать наваять трекер.пхп (в принципе я тоже на пыхе не писец).

]]> Re:Прием патчей к исходникам LOR-NG https://rulinux.net/message.php?newsid=1588&page=1#12572 https://rulinux.net/message.php?newsid=1588&page=1#12572 Tue, 12 May 2009 17:46:28 +0400 > Нечего оправдываться и переводить стрелки на туксойда - никто же не говорю, что разработчиков надо убить прямо сейчас :)

Я говорю про то что времени не было их досканально просмотреть :-)

> это не оптимизация а то, что хотелось бы поиметь

пд оптимизацией я имел в виду минимизацию вызовов к СУБД

> Вообще, было бы неплохо если бы разрабы взяли бы за правило складывать сорцы в конце дня куда-нить в ФС сервера, раз уж cvs пока нет.

Да погоди, свн скоро будет уже

]]> Re:Прием патчей к исходникам LOR-NG https://rulinux.net/message.php?newsid=1588&page=1#12576 https://rulinux.net/message.php?newsid=1588&page=1#12576 Tue, 12 May 2009 09:46:02 +0400 Нечего оправдываться и переводить стрелки на туксойда - никто же не говорю, что разработчиков надо убить прямо сейчас :)

К тому же таймстампы, актуальные сорцы и тестовые данные - это не оптимизация а то, что хотелось бы поиметь, что бы тракер.пхп пририсовать.

Вообще, было бы неплохо если бы разрабы взяли бы за правило складывать сорцы в конце дня куда-нить в ФС сервера, раз уж cvs пока нет.

]]> Re:Прием патчей к исходникам LOR-NG https://rulinux.net/message.php?newsid=1588&page=1#12575 https://rulinux.net/message.php?newsid=1588&page=1#12575 Tue, 12 May 2009 05:08:40 +0400 форум Туксойд писал, я его исходники хоть и правил, но не оптимизировал

]]> Re:Прием патчей к исходникам LOR-NG https://rulinux.net/message.php?newsid=1588&page=1#12574 https://rulinux.net/message.php?newsid=1588&page=1#12574 Tue, 12 May 2009 03:45:41 +0400 А можешь в forum.php заменить последовательность действий:

$ath = mysql_query("INSERT INTO forum_threads(name, user_name, posting_date, forum_id) VALUES('$title', '$username', '$posting_date', '$forumid')");

...

$tbl = mysql_query("SELECT thread_id FROM forum_threads WHERE name = '$title' AND posting_date = '$posting_date' AND forum_id = '$forumid'");

...

$thr = mysql_fetch_array($tbl);

$thread_id = $thr['thread_id'];

на:

$ath = mysql_query("INSERT INTO forum_....

$thread_id = mysql_insert_id($ath);

И не понял для чего после создания треда повторно брать дату для сообщения, а потом апдейтить тред новой датой. Наверное это можно закоментить нафиг.

Вообще, хотелось бы в базе иметь timestamp'ы и даты в полях типа timestamp а не в виде строки или целого. Можешь добавить колонку created_at типа timestamp в таблицы news, comments, forum_messages, forum_threads etc? Они будут автоматически инициализироваться текущим временем при создании записи.

И ещё хотелось бы поиметь полный набор сорцов, включая файлик view-message.php и х/з что там ещё было пропущено. Тестовый набор данных был бы тоже не вреден.

]]> Re:Прием патчей к исходникам LOR-NG https://rulinux.net/message.php?newsid=1588&page=1#12571 https://rulinux.net/message.php?newsid=1588&page=1#12571 Mon, 11 May 2009 17:59:31 +0400 Я пытался унифицировать вывод ошибок функциями

]]> Re:Прием патчей к исходникам LOR-NG https://rulinux.net/message.php?newsid=1588&page=1#12570 https://rulinux.net/message.php?newsid=1588&page=1#12570 Mon, 11 May 2009 17:25:30 +0400 > мало ли где понадобится его проверка

-1 там говорит, что не нашлось строк в наборе данных. То же самое скажет и пустой массив. Если понадобится проверка - можно просто будет проверить длину массива.

]]> Re:Прием патчей к исходникам LOR-NG https://rulinux.net/message.php?newsid=1588&page=1#12569 https://rulinux.net/message.php?newsid=1588&page=1#12569 Mon, 11 May 2009 17:11:25 +0400 > 1. Повыбрасывать из кода в classes/*.class.php вызовы "mysql_close();"

Принято. Начинал писать достаточно сумбурно, а сейчас все руки не доходят подобный мусор выкинуть

> 2. Из метода pages::get_meta_data (classes/pages.class.php) следующие строки:

ИМХО, лучше оставить return, mysql_close естественно выбросить. А тут - мало ли где понадобится его проверка

]]> Прием патчей к исходникам LOR-NG https://rulinux.net/message.php?newsid=1588&page=1#12568 https://rulinux.net/message.php?newsid=1588&page=1#12568 Mon, 11 May 2009 14:03:12 +0400 Раз уж исходники стали более-менее доступны, предлагаю прикрепить в этом разделе тему с предложениями по их улучшению. Для затравки два предложения:

1. Повыбрасывать из кода в classes/*.class.php вызовы "mysql_close();" - насколько я понимаю, из-за этих вызовов каждый вызов метода класса будет переустанавливать соединение к БД.

Да и пролог, устанавливающий соединение к БД, выбирающий зачем-то базу и устанавливающий кодировку для сессии - лучше куда-нить в глобальный инклюд вытащить и делать это всё однократно при инициализации обработки запроса, чем на каждый вызов метода.

2. Из метода pages::get_meta_data (classes/pages.class.php) следующие строки:

  2. if (mysql_numrows($meta_res)<=0){
  3. mysql_close();return -1;
  4. }
ибо возвращённое значение нигде не проверяется (и при пустом наборе данных даёт ошибки при использовании в foreach, например в incs/header.inc.php), а пустой массив - вполне адкватная репрезентация пустого набора данных

]]>