/[LeafOK_CVS]/lbbs/src/article_favor.c
ViewVC logotype

Diff of /lbbs/src/article_favor.c

Parent Directory Parent Directory | Revision Log Revision Log | View Patch Patch

Revision 1.3 by sysadm, Wed Oct 15 03:02:17 2025 UTC Revision 1.6 by sysadm, Wed Oct 29 01:48:46 2025 UTC
# Line 148  int article_favor_save_inc(const ARTICLE Line 148  int article_favor_save_inc(const ARTICLE
148                           "DELETE FROM article_favorite WHERE UID = %d AND AID IN (",                           "DELETE FROM article_favorite WHERE UID = %d AND AID IN (",
149                           p_favor->uid);                           p_favor->uid);
150    
151          for (i = 0, j = 0; i < p_favor->aid_base_cnt && j < p_favor->aid_inc_cnt;)          for (i = 0, j = 0; j < p_favor->aid_inc_cnt;)
152          {          {
153                  if (p_favor->aid_base[i] == p_favor->aid_inc[j]) // XOR - delete record                  if (i < p_favor->aid_base_cnt && p_favor->aid_base[i] == p_favor->aid_inc[j]) // XOR - delete record
154                  {                  {
155                          snprintf(tuple_tmp, sizeof(tuple_tmp), "%d, ", p_favor->aid_inc[j]);                          snprintf(tuple_tmp, sizeof(tuple_tmp), "%d, ", p_favor->aid_inc[j]);
156                          strncat(sql_del, tuple_tmp, sizeof(sql_del) - 1 - strnlen(sql_del, sizeof(sql_del)));                          strncat(sql_del, tuple_tmp, sizeof(sql_del) - 1 - strnlen(sql_del, sizeof(sql_del)));
# Line 159  int article_favor_save_inc(const ARTICLE Line 159  int article_favor_save_inc(const ARTICLE
159                          i++;                          i++;
160                          j++;                          j++;
161                  }                  }
162                  else if (p_favor->aid_base[i] < p_favor->aid_inc[j]) // skip existing record                  else if (i < p_favor->aid_base_cnt && p_favor->aid_base[i] < p_favor->aid_inc[j]) // skip existing record
163                  {                  {
164                          i++;                          i++;
165                  }                  }
166                  else // if (p_favor->aid_base[i] > p_favor->aid_inc[j])                  else // if (i >= p_favor->aid_base_cnt || p_favor->aid_base[i] > p_favor->aid_inc[j])
167                  {                  {
168                          snprintf(tuple_tmp, sizeof(tuple_tmp),                          snprintf(tuple_tmp, sizeof(tuple_tmp),
169                                           "(%d, %d), ",                                           "(%d, %d), ",
# Line 321  int article_favor_check(int32_t aid, con Line 321  int article_favor_check(int32_t aid, con
321                          mid = (left + right) / 2;                          mid = (left + right) / 2;
322                          if (aid < (i == 0 ? p_favor->aid_base[mid] : p_favor->aid_inc[mid]))                          if (aid < (i == 0 ? p_favor->aid_base[mid] : p_favor->aid_inc[mid]))
323                          {                          {
324                                  right = mid;                                  right = mid - 1;
325                          }                          }
326                          else if (aid > (i == 0 ? p_favor->aid_base[mid] : p_favor->aid_inc[mid]))                          else if (aid > (i == 0 ? p_favor->aid_base[mid] : p_favor->aid_inc[mid]))
327                          {                          {
# Line 372  int article_favor_set(int32_t aid, ARTIC Line 372  int article_favor_set(int32_t aid, ARTIC
372                          mid = (left + right) / 2;                          mid = (left + right) / 2;
373                          if (aid < (i == 0 ? p_favor->aid_base[mid] : p_favor->aid_inc[mid]))                          if (aid < (i == 0 ? p_favor->aid_base[mid] : p_favor->aid_inc[mid]))
374                          {                          {
375                                  right = mid;                                  right = mid - 1;
376                          }                          }
377                          else if (aid > (i == 0 ? p_favor->aid_base[mid] : p_favor->aid_inc[mid]))                          else if (aid > (i == 0 ? p_favor->aid_base[mid] : p_favor->aid_inc[mid]))
378                          {                          {
# Line 508  int query_favor_articles(ARTICLE_FAVOR * Line 508  int query_favor_articles(ARTICLE_FAVOR *
508                          return -3;                          return -3;
509                  }                  }
510    
511                  // acquire lock of section                  if (get_section_info(p_section, p_snames[i], NULL, NULL) < 0)
                 if (section_list_rd_lock(p_section) < 0)  
                 {  
                         log_error("section_list_rd_lock(sid = %d) error\n", p_section->sid);  
                         return -4;  
                 }  
   
                 memcpy(p_snames[i], p_section->sname, sizeof(p_snames[i]));  
   
                 // release lock of section  
                 if (section_list_rd_unlock(p_section) < 0)  
512                  {                  {
513                          log_error("section_list_rd_unlock(sid = %d) error\n", p_section->sid);                          log_error("get_section_info(sid=%d) error\n", p_section->sid);
514                          return -4;                          return -4;
515                  }                  }
516          }          }

Legend:
Removed lines/characters  
Changed lines/characters
  Added lines/characters
webmaster@leafok.com
 ViewVC Help
Powered by ViewVC 1.3.0-beta1